feat!: run on Node 24 runtime and upgrade dependencies#208
Merged
Voziv merged 2 commits intoMay 23, 2026
mainargocd-diff-action/argocd-diff-action:mainfrom
worktree-dependency-upgradesargocd-diff-action/argocd-diff-action:worktree-dependency-upgradesCopy head branch name to clipboard
Merged
feat!: run on Node 24 runtime and upgrade dependencies#208Voziv merged 2 commits intomainargocd-diff-action/argocd-diff-action:mainfrom worktree-dependency-upgradesargocd-diff-action/argocd-diff-action:worktree-dependency-upgradesCopy head branch name to clipboard
Voziv merged 2 commits into
mainargocd-diff-action/argocd-diff-action:mainfrom
worktree-dependency-upgradesargocd-diff-action/argocd-diff-action:worktree-dependency-upgradesCopy head branch name to clipboard
Conversation
Bump to latest: @actions/github, @eslint/js (10), eslint (10), globals (17), @types/sinon (21), sinon (22), pre-commit (2), semantic-release (25), typescript (6). Align @types/node to the Node 24 runtime. Remove eslint-plugin-github, eslint-plugin-import and @eslint/eslintrc: none were imported by eslint.config.mjs (eslint-plugin-import was registered with no active import/* rules), so dropping them clears the eslint 10 peer-dependency warnings without changing lint behaviour.
Switch the action runtime from node20 to node24 and bump .nvmrc to match. Update CLAUDE.md references accordingly. BREAKING CHANGE: the action now requires a runner with Node 24 available. GitHub-hosted runners provide it; self-hosted runners must have Node 24 installed.
Coverage report
Test suite run success28 tests passing in 4 suites. Report generated by 🧪jest coverage report action from bacdfa1 |
argocd-diff-action-bot Bot
pushed a commit
that referenced
this pull request
May 24, 2026
## [0.6.12](0.6.11...0.6.12) (2026-05-24) ### ⚠ BREAKING CHANGES * run on Node 24 runtime and upgrade dependencies (#208) ### Build & Dependencies * **deps-dev:** upgrade pnpm to 11.2.2 ([#207](#207)) ([f69a1e6](f69a1e6)) ### Documentation * add CLAUDE.md ([664ab77](664ab77)) ### Features * run on Node 24 runtime and upgrade dependencies ([#208](#208)) ([13477ad](13477ad)), closes [#175](#175) ### Bug Fixes * use dynamic timezone abbreviation in PR comment timestamp ([#209](#209)) ([81f8e40](81f8e40)) [skip release]
Contributor
|
🎉 This PR is included in version 0.6.12 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
@actions/github,@eslint/js(10),eslint(10),globals(17),@types/sinon(21),sinon(22),pre-commit(2),semantic-release(25),typescript(6).node20tonode24and align.nvmrc+@types/node(pinned to 24.x) to match. The dev toolchain already required Node ≥ 22.13 (pnpm 11.2.2, semantic-release 25), so.nvmrc: 20was stale.eslint-plugin-github,eslint-plugin-importand@eslint/eslintrc. None were imported byeslint.config.mjs(eslint-plugin-importwas registered but had no activeimport/*rules), and dropping them clears the eslint 10 peer-dependency warnings with no change to lint behaviour.@types/nodeis intentionally held at 24.x (not 25) to match the runtime; dependabot's existing "ignore @types/node major" rule keeps it pinned there.dist/is left untouched — it is regenerated by the release workflow.BREAKING CHANGE
The action now runs on the Node 24 runtime instead of Node 20. It requires a runner with Node 24 available. GitHub-hosted runners provide it; self-hosted runners must have Node 24 installed or the action will fail to start.
Test plan
pnpm run lintpasses (eslint 10)pnpm run buildsucceeds (ncc + typescript 6)pnpm run test— 27/28 pass; the one failure (lib.test.tsshell-error assertion) is pre-existing and environment-specific (local/bin/shis bash, not dash) and passes on CI/UbuntuChecksworkflow green on this PR