Skip to content

chore(deps): bump actions/upload-artifact from 4 to 6#147

Merged
kayodebristol merged 2 commits intomainfrom
dependabot/github_actions/actions/upload-artifact-6
Feb 4, 2026
Merged

chore(deps): bump actions/upload-artifact from 4 to 6#147
kayodebristol merged 2 commits intomainfrom
dependabot/github_actions/actions/upload-artifact-6

Conversation

@dependabot
Copy link
Contributor

@dependabot dependabot bot commented on behalf of github Feb 2, 2026

Bumps actions/upload-artifact from 4 to 6.

Release notes

Sourced from actions/upload-artifact's releases.

v6.0.0

v6 - What's new

[!IMPORTANT] actions/upload-artifact@v6 now runs on Node.js 24 (runs.using: node24) and requires a minimum Actions Runner version of 2.327.1. If you are using self-hosted runners, ensure they are updated before upgrading.

Node.js 24

This release updates the runtime to Node.js 24. v5 had preliminary support for Node.js 24, however this action was by default still running on Node.js 20. Now this action by default will run on Node.js 24.

What's Changed

Full Changelog: actions/upload-artifact@v5.0.0...v6.0.0

v5.0.0

What's Changed

BREAKING CHANGE: this update supports Node v24.x. This is not a breaking change per-se but we're treating it as such.

New Contributors

Full Changelog: actions/upload-artifact@v4...v5.0.0

v4.6.2

What's Changed

New Contributors

Full Changelog: actions/upload-artifact@v4...v4.6.2

v4.6.1

What's Changed

... (truncated)

Commits
  • b7c566a Merge pull request #745 from actions/upload-artifact-v6-release
  • e516bc8 docs: correct description of Node.js 24 support in README
  • ddc45ed docs: update README to correct action name for Node.js 24 support
  • 615b319 chore: release v6.0.0 for Node.js 24 support
  • 017748b Merge pull request #744 from actions/fix-storage-blob
  • 38d4c79 chore: rebuild dist
  • 7d27270 chore: add missing license cache files for @​actions/core, @​actions/io, and mi...
  • 5f643d3 chore: update license files for @​actions/artifact@​5.0.1 dependencies
  • 1df1684 chore: update package-lock.json with @​actions/artifact@​5.0.1
  • b5b1a91 fix: update @​actions/artifact to ^5.0.0 for Node.js 24 punycode fix
  • Additional commits viewable in compare view

Dependabot compatibility score

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot show <dependency name> ignore conditions will show all of the ignore conditions of the specified dependency
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

Bumps [actions/upload-artifact](https://github.com/actions/upload-artifact) from 4 to 6.
- [Release notes](https://github.com/actions/upload-artifact/releases)
- [Commits](actions/upload-artifact@v4...v6)

---
updated-dependencies:
- dependency-name: actions/upload-artifact
  dependency-version: '6'
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
@dependabot @github
Copy link
Contributor Author

dependabot bot commented on behalf of github Feb 2, 2026

Labels

The following labels could not be found: dependencies, github-actions. Please create them before Dependabot can add them to a pull request.

Please fix the above issues or remove invalid values from dependabot.yml.

@kayodebristol
Copy link
Contributor

⚠️ Dev Lead: Check Failure detected (Node 18.x). @github-copilot please analyze logs and fix the test suite.

@kayodebristol
Copy link
Contributor

⚠️ Dev Lead: Checks failed. @github-copilot please analyze logs and fix.

@kayodebristol
Copy link
Contributor

@copilot please address the failing checks

Copy link
Contributor

Copilot AI commented Feb 3, 2026

@kayodebristol I've opened a new pull request, #148, to work on those changes. Once the pull request is ready, I'll request review from you.

@kayodebristol
Copy link
Contributor

@copilot please address the failing checks on this PR.\n\nCI is failing during with:\n- Unknown command: "error"

To see a list of supported npm commands, run:
npm help\n- \n\nThis suggests the workflow is using an npm version that does not support (or otherwise not using the intended package manager for this repo).\n\nPlease:\n1) Identify the repo’s intended package manager/tooling (npm vs pnpm vs yarn) and ensure the workflow uses a compatible version.\n - If using pnpm/yarn workspaces, enable Corepack and install with the correct tool.\n - If using npm workspaces, ensure Node/setup yields npm >=7 and lockfile matches.\n2) Update so Node 18/20 jobs both install deps successfully (or adjust the matrix if Node 18 is not supported).\n3) Re-run CI and confirm green for the workflow jobs.\n\nThanks!

Copy link
Contributor

Copilot AI commented Feb 3, 2026

@kayodebristol I've opened a new pull request, #149, to work on those changes. Once the pull request is ready, I'll request review from you.

@kayodebristol
Copy link
Contributor

@copilot please address the failing checks on this PR.

CI is failing during "Install dependencies" with:

  • npm error code EUNSUPPORTEDPROTOCOL
  • Unsupported URL Type "workspace:": workspace:*

This suggests the workflow is using an npm version that does not support workspace:* (or otherwise not using the intended package manager for this repo).

Please:

  1. Identify the repo’s intended package manager/tooling (npm vs pnpm vs yarn) and ensure the workflow uses a compatible version.
    • If using pnpm/yarn workspaces, enable Corepack and install with the correct tool.
    • If using npm workspaces, ensure setup yields npm >=7 and lockfile matches.
  2. Update .github/workflows so Node 18/20 jobs both install deps successfully (or adjust the matrix if Node 18 is not supported).
  3. Re-run CI and confirm green for the CI workflow jobs.

Copy link
Contributor

Copilot AI commented Feb 3, 2026

@kayodebristol I've opened a new pull request, #150, to work on those changes. Once the pull request is ready, I'll request review from you.

@kayodebristol
Copy link
Contributor

@copilot CI is failing on Node.js 18.x (see 'Test on Node.js (18.x)' in Actions). Please investigate and push a fix to this PR so all checks go green. After this merges, we can delete/close the stacked draft sub-PRs (#148/#149/#150) that target this branch.

Copy link
Contributor

Copilot AI commented Feb 3, 2026

@kayodebristol I've opened a new pull request, #151, to work on those changes. Once the pull request is ready, I'll request review from you.

@kayodebristol
Copy link
Contributor

Automated sweep: checks are not green (BAD:Test on Node.js (18.x):COMPLETED/FAILURE,Test on Node.js (20.x):COMPLETED/CANCELLED). [@copilot] please diagnose and fix CI/test failures for this PR (prefer minimal changes).

Copy link
Contributor

Copilot AI commented Feb 3, 2026

@kayodebristol I've opened a new pull request, #152, to work on those changes. Once the pull request is ready, I'll request review from you.

@kayodebristol
Copy link
Contributor

CI is not green for this PR.\n\nChecks output (summary):\n\n\n\n@copilot please:\n1) Reproduce the failing check locally if possible (or inspect failing workflow logs).\n2) Push a fix to this PR branch so all required checks pass.\n3) Reply here with what you changed + why.\n\nDo not merge until checks are green.

Copy link
Contributor

Copilot AI commented Feb 3, 2026

@kayodebristol I've opened a new pull request, #153, to work on those changes. Once the pull request is ready, I'll request review from you.

@kayodebristol
Copy link
Contributor

CI is not green for this PR.

Checks output (summary):

Test on Node.js (18.x)	fail	10s	https://github.com/plures/praxis/actions/runs/21610552467/job/62278069803	
CodeQL	pass	1s	https://github.com/plures/praxis/runs/62278160379	
Test on Node.js (20.x)	fail	0	https://github.com/plures/praxis/actions/runs/21610552467/job/62278069768	
Analyze (javascript)	pass	1m11s	https://github.com/plures/praxis/actions/runs/21610552483/job/62278069693	
Build Check (C#)	pass	42s	https://github.com/plures/praxis/actions/runs/21610552478/job/62278069696	
Deno Compatibility Check	pass	12s	https://github.com/plures/praxis/actions/runs/21610552467/job/62278069764	
Test C# on .NET 8	pass	52s	https://github.com/plures/praxis/actions/runs/21610552467/job/62278069763	

@copilot please:

  1. Reproduce the failing check locally if possible (or inspect failing workflow logs).
  2. Push a fix to this PR branch so all required checks pass.
  3. Reply here with what you changed + why.

Do not merge until checks are green.

Copy link
Contributor

Copilot AI commented Feb 3, 2026

@kayodebristol I've opened a new pull request, #154, to work on those changes. Once the pull request is ready, I'll request review from you.

@kayodebristol
Copy link
Contributor

@copilot CI is red: Test on Node.js (18.x) + (20.x) failing. Local diff is minimal (actions/upload-artifact bump only), so this is likely an existing test/tooling failure exposed by the run. Please open the failing job logs, identify the root error, and push a fix (prefer a new commit on this PR or a follow-up PR if needed). Do not merge until green.

Copy link
Contributor

Copilot AI commented Feb 3, 2026

@kayodebristol I've opened a new pull request, #155, to work on those changes. Once the pull request is ready, I'll request review from you.

@kayodebristol
Copy link
Contributor

CI is currently failing on Node 18 (Test on Node.js (18.x)), so per policy we won’t merge yet.

We have a consolidated fix in #155 (switch CI to pnpm + workspace:* support). Plan:

@kayodebristol
Copy link
Contributor

@copilot Please fix failing CI for this PR.

Steps:

  1. Pull branch locally (or use repo context) and reproduce the failing checks.
  2. Fix the root cause (tests/lint/build).
  3. Push updates to this PR branch and ensure all required checks are green.

Failing checks summary (from gh pr checks --required):

no checks reported on the 'copilot/sub-pr-147-05ed7eb7-cfa8-48b8-bd81-859a19451045' branch
no required checks reported on the 'dependabot/github_actions/actions/upload-artifact-6' branch

When done, comment what changed and why.

Copy link
Contributor

Copilot AI commented Feb 3, 2026

@kayodebristol I've opened a new pull request, #156, to work on those changes. Once the pull request is ready, I'll request review from you.

* Initial plan

* fix: switch from npm to pnpm for workspace support

- Add packageManager field to package.json (pnpm@9.15.1)
- Create pnpm-workspace.yaml for monorepo configuration
- Update CI workflow to use pnpm instead of npm
- Enable Corepack in CI workflow
- Remove package-lock.json, add pnpm-lock.yaml
- Fixes EUNSUPPORTEDPROTOCOL error with workspace:* dependencies

Co-authored-by: kayodebristol <3579196+kayodebristol@users.noreply.github.com>

* fix: resolve TypeScript typecheck errors

- Remove unused import 'path' from conversations.ts
- Add null check for classification.confidence
- Filter undefined values from labels array
- Prefix unused parameter with underscore in gateNotDuplicate
- Explicitly type 'passed' as boolean in gateHasMetadata
- Prefix unused parameter with underscore in normalizeCodeBlocks

Fixes all 6 TypeScript errors blocking CI

Co-authored-by: kayodebristol <3579196+kayodebristol@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: kayodebristol <3579196+kayodebristol@users.noreply.github.com>
@kayodebristol kayodebristol merged commit b21b3f2 into main Feb 4, 2026
@kayodebristol kayodebristol deleted the dependabot/github_actions/actions/upload-artifact-6 branch February 4, 2026 00:01
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