Skip to content

Fix CI by upgrading actions/upload-artifact from v3 to v4#83

Merged
jakubtobiasz merged 3 commits intoCommerceWeavers:mainfrom
jakubtobiasz:claude/review-pr-82-011CUNWy3TZ5rBVPTBuMLEzm
Oct 22, 2025
Merged

Fix CI by upgrading actions/upload-artifact from v3 to v4#83
jakubtobiasz merged 3 commits intoCommerceWeavers:mainfrom
jakubtobiasz:claude/review-pr-82-011CUNWy3TZ5rBVPTBuMLEzm

Conversation

@jakubtobiasz
Copy link
Member

The deprecated v3 of actions/upload-artifact was causing CI failures. Updated both occurrences in the workflow file to use v4.

🤖 Generated with Claude Code

The deprecated v3 of actions/upload-artifact was causing CI failures.
Updated both occurrences in the workflow file to use v4.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
Node.js 18.x reached End-of-Life and is no longer supported by GitHub Actions.
Updated both job matrices to use Node 20.x (current LTS) and added the
node_version parameter to BuildTestAppAction calls to ensure the correct
version is used for e2e JavaScript tests.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
The previous version 1.0.1 only supported Node.js ^14 || ^16 || ^18, causing
the build to fail with Node 20.x. Version 1.0.3 (used in Sylius 1.13) adds
support for Node.js ^18 || ^20, making it compatible with our CI configuration.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
@jakubtobiasz jakubtobiasz merged commit 968255d into CommerceWeavers:main Oct 22, 2025
8 checks passed
@jakubtobiasz jakubtobiasz deleted the claude/review-pr-82-011CUNWy3TZ5rBVPTBuMLEzm branch October 22, 2025 16:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants