Skip to content

Comments

ci: migrate from addnab/docker-run-action to tj-actions/docker-run#1078

Merged
Brooooooklyn merged 3 commits intomainfrom
copilot/migrate-docker-run-action
Feb 17, 2026
Merged

ci: migrate from addnab/docker-run-action to tj-actions/docker-run#1078
Brooooooklyn merged 3 commits intomainfrom
copilot/migrate-docker-run-action

Conversation

Copy link
Contributor

Copilot AI commented Feb 16, 2026

CI failed after migrating from addnab/docker-run-action to tj-actions/docker-run. The new action appends args directly to docker run, unlike the old action which used --entrypoint=sh -c. Shell built-ins like set -e fail with "executable file not found".

Changed three failing job steps:

- args: |
-   set -e
-   yarn test -s
-   ls -la
+ args: sh -c "set -e && yarn test -s && ls -la"

Jobs with single-line commands (no shell built-ins) required no changes.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • https://api.github.com/repos/napi-rs/node-rs/commits/723ad5951443f8c49af62563b509033a9b1bba0a/check-runs
    • Triggering command: /usr/bin/curl curl -s REDACTED (http block)
  • https://api.github.com/repos/napi-rs/node-rs/commits/723ad5951443f8c49af62563b509033a9b1bba0a/status
    • Triggering command: /usr/bin/curl curl -s -H Accept: application/vnd.github.v3+json REDACTED (http block)

If you need me to access, download, or install something from one of these locations, you can either:


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

- Updated all 5 usages in .github/workflows/ci.yaml
- Changed action from addnab/docker-run-action@v3 to tj-actions/docker-run@v2
- Renamed 'run' parameter to 'args'
- Added required 'name' parameter with unique identifiers for each container
- Container names include matrix node version for uniqueness across jobs

Co-authored-by: Brooooooklyn <3468483+Brooooooklyn@users.noreply.github.com>
Copilot AI changed the title [WIP] Migrate deprecated addnab/docker-run-action to tj-actions/docker-run Migrate from deprecated addnab/docker-run-action to tj-actions/docker-run Feb 16, 2026
Copilot AI requested a review from Brooooooklyn February 16, 2026 15:21
Co-authored-by: Brooooooklyn <3468483+Brooooooklyn@users.noreply.github.com>
Copilot AI changed the title Migrate from deprecated addnab/docker-run-action to tj-actions/docker-run Fix docker-run migration: wrap multi-line commands with sh -c Feb 16, 2026
@Brooooooklyn Brooooooklyn marked this pull request as ready for review February 16, 2026 16:20
@Brooooooklyn Brooooooklyn changed the title Fix docker-run migration: wrap multi-line commands with sh -c ci: migrate from addnab/docker-run-action to tj-actions/docker-run Feb 17, 2026
@Brooooooklyn Brooooooklyn merged commit 2d2a046 into main Feb 17, 2026
44 checks passed
@Brooooooklyn Brooooooklyn deleted the copilot/migrate-docker-run-action branch February 17, 2026 01:16
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