Skip to content

Bump tj-actions/changed-files v46 → v47 for Node.js 24 compatibility#26

Merged
michieldegezelle merged 1 commit into
mainfrom
upgrade/tj-actions-node24
Jun 3, 2026
Merged

Bump tj-actions/changed-files v46 → v47 for Node.js 24 compatibility#26
michieldegezelle merged 1 commit into
mainfrom
upgrade/tj-actions-node24

Conversation

@michieldegezelle
Copy link
Copy Markdown
Collaborator

Summary

  • Bumps tj-actions/changed-files from v46 to v47 across run_tests.yml, check_tests.yml, and update_templates_production.yml
  • v47 upgrades the action's internal runtime to Node.js 24, resolving the deprecation warning introduced after the previous PR was merged

Test plan

  • Verify no deprecation warnings appear in the check-changed-templates job after merge
  • Confirm run-tests workflow behaves as before on a PR with changed templates

🤖 Generated with Claude Code

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Jun 3, 2026

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 7391b2f4-af9c-49ba-b16b-f32834892378

📥 Commits

Reviewing files that changed from the base of the PR and between db59900 and b9cfb82.

📒 Files selected for processing (3)
  • .github/workflows/check_tests.yml
  • .github/workflows/run_tests.yml
  • .github/workflows/update_templates_production.yml
🚧 Files skipped from review as they are similar to previous changes (3)
  • .github/workflows/run_tests.yml
  • .github/workflows/check_tests.yml
  • .github/workflows/update_templates_production.yml

Walkthrough

Three GitHub workflow files bump the tj-actions/changed-files action from v46 to v47 for consistent file-change detection across check-tests, run-tests, and production template-update pipelines. No logic, inputs, or workflow behavior changed.

Changes

Action version bump

Layer / File(s) Summary
Update tj-actions/changed-files to v47
.github/workflows/check_tests.yml, .github/workflows/run_tests.yml, .github/workflows/update_templates_production.yml
Three workflows each update the tj-actions/changed-files action reference from @v46 to @v47 (one via a pinned commit SHA). No workflow logic, file-detection inputs, conditional test selection, or deployment behavior changed.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Description check ❓ Inconclusive The description includes a summary of changes and test plan, but omits the required 'Type of change' and 'Checklist' sections from the template. Add the 'Type of change' section (e.g., selecting 'Bug fix') and complete the 'Checklist' section to fully match the repository template.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the main change: upgrading tj-actions/changed-files from v46 to v47, with a specific reason (Node.js 24 compatibility).
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch upgrade/tj-actions-node24

Comment @coderabbitai help to get the list of available commands and usage tips.

@michieldegezelle michieldegezelle self-assigned this Jun 3, 2026
Copy link
Copy Markdown

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In @.github/workflows/check_tests.yml:
- Line 24: Replace the mutable action tag tj-actions/changed-files@v47 in the
workflow with the immutable commit SHA currently pointed to by that tag
(24d32ffd492484c1d75e0c0b894501ddb9d30d62) so the workflow references a fixed
commit; update the usage of tj-actions/changed-files@v47 to
tj-actions/changed-files@24d32ffd492484c1d75e0c0b894501ddb9d30d62 to pin the
action.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 2710c27b-2d15-4be2-aedd-2035f76db920

📥 Commits

Reviewing files that changed from the base of the PR and between b0ce7d1 and db59900.

📒 Files selected for processing (3)
  • .github/workflows/check_tests.yml
  • .github/workflows/run_tests.yml
  • .github/workflows/update_templates_production.yml

Comment thread .github/workflows/check_tests.yml Outdated
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@michieldegezelle michieldegezelle force-pushed the upgrade/tj-actions-node24 branch from db59900 to b9cfb82 Compare June 3, 2026 13:10
@michieldegezelle michieldegezelle merged commit d31f61b into main Jun 3, 2026
7 checks passed
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.

1 participant