Skip to content

build(dev): test node.js CI linux builds with newer docker version#264

Merged
2bndy5 merged 1 commit intomainfrom
update-node-ci
Feb 18, 2026
Merged

build(dev): test node.js CI linux builds with newer docker version#264
2bndy5 merged 1 commit intomainfrom
update-node-ci

Conversation

@2bndy5
Copy link
Copy Markdown
Collaborator

@2bndy5 2bndy5 commented Feb 18, 2026

Replaces addnab/docker-run-action with tj-actions/docker-run

Apparently, addnab/docker-run-action is no longer maintained; see addnab/docker-run-action#62.

Summary by CodeRabbit

  • Chores
    • Updated build workflow infrastructure to use an improved Docker execution action.
    • Updated spell-check configuration.

Replaces [addnab/docker-run-action] with [tj-actions/docker-run]

Apparently, [addnab/docker-run-action] is no longer maintained; see
addnab/docker-run-action#62.

[addnab/docker-run-action]: https://github.com/addnab/docker-run-action
[tj-actions/docker-run]: https://github.com/tj-actions/docker-run
@2bndy5 2bndy5 added dependencies Pull requests that update a dependency file github_actions Pull requests that update GitHub Actions code labels Feb 18, 2026
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Feb 18, 2026

No actionable comments were generated in the recent review. 🎉


Walkthrough

Updated GitHub Actions workflow to replace the Docker run action from addnab/docker-run-action to tj-actions/docker-run@v2.2.1, modified test command invocation syntax from run: to args:, and removed the now-obsolete "addnab" word from the spell checker configuration.

Changes

Cohort / File(s) Summary
GitHub Actions Workflow
.github/workflows/node-js-packaging.yml
Replaced docker-run action to tj-actions/docker-run@v2.2.1, added dynamic test step name using matrix variable, and changed test command invocation from run: yarn test to args: yarn test.
Spell Checker Configuration
cspell.config.yml
Removed "addnab" from the words allowlist.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main change: updating the CI workflow to use a newer docker action (tj-actions/docker-run instead of addnab/docker-run-action) for Linux builds.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch update-node-ci

Tip

Issue Planner is now in beta. Read the docs and try it out! Share your feedback on Discord.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

@2bndy5 2bndy5 merged commit f71cfad into main Feb 18, 2026
38 checks passed
@2bndy5 2bndy5 deleted the update-node-ci branch February 18, 2026 00:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file github_actions Pull requests that update GitHub Actions code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant