fix: update upload-artifact action to version 4

[ErikBjare]

Important

Update upload-artifact action to major version v4 in nodejs.yml for consistency across jobs.

• GitHub Actions:
  • Update upload-artifact action from v4.3.6 to v4 in nodejs.yml.
  • Affects Upload build steps in build-webpack and build-vite jobs.
  • Affects Upload logs step in test job.
  • Affects Upload screenshots step in test job.

^{This description was created by for 710fa62. It will automatically update as commits are pushed.}

[ellipsis-dev]

👍 Looks good to me! Reviewed everything up to 710fa62 in 50 seconds

More details

• Looked at 32 lines of code in 1 files
• Skipped 0 files when reviewing.
• Skipped posting 7 drafted comments based on config settings.

1. .github/workflows/nodejs.yml:32

• Draft comment:
  Using unpinned v4 for upload-artifact may fetch unexpected updates. Consider pinning the patch version.
• Reason this comment was not posted:
  Confidence changes required: 33% <= threshold 50%
  None

2. .github/workflows/nodejs.yml:60

• Draft comment:
  Consider pinning upload-artifact to a specific patch version instead of 'v4' to ensure consistent behavior.
• Reason this comment was not posted:
  Confidence changes required: 33% <= threshold 50%
  None

3. .github/workflows/nodejs.yml:194

• Draft comment:
  Unpinned version in logs upload step might lead to unexpected updates. Pin a specific patch version if stability is required.
• Reason this comment was not posted:
  Confidence changes required: 33% <= threshold 50%
  None

4. .github/workflows/nodejs.yml:32

• Draft comment:
  Updated upload-artifact version to 'v4'. Consider pinning to a specific minor release (e.g., v4.3.x) to avoid unintentional breaking changes.
• Reason this comment was not posted:
  Confidence changes required: 50% <= threshold 50%
  None

5. .github/workflows/nodejs.yml:60

• Draft comment:
  Updated upload-artifact version to 'v4'. Consider pinning to a specific minor release to ensure stability.
• Reason this comment was not posted:
  Confidence changes required: 50% <= threshold 50%
  None

6. .github/workflows/nodejs.yml:194

• Draft comment:
  Updated upload-artifact version and modified artifact name to include matrix values. Verify downstream consumers use the new name.
• Reason this comment was not posted:
  Confidence changes required: 50% <= threshold 50%
  None

7. .github/workflows/nodejs.yml:175

• Draft comment:
  The version for the 'actions/upload-artifact' in the 'Upload screenshots' step is still set to v4.3.6, while the other occurrences have been updated to v4. For consistency, please update it to v4.
• Reason this comment was not posted:
  Comment was not on a location in the diff, so it can't be submitted as a review comment.

Workflow ID: wflow_WFfdmQPO5RyHAhMI

---

You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 25.98%. Comparing base (8679b69) to head (710fa62).
Report is 3 commits behind head on master.

Additional details and impacted files

@@           Coverage Diff           @@
##           master     #641   +/-   ##
=======================================
  Coverage   25.98%   25.98%           
=======================================
  Files          27       27           
  Lines        1643     1643           
  Branches      292      292           
=======================================
  Hits          427      427           
  Misses       1157     1157           
  Partials       59       59           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.