Skip to content

[nightshift] ci-fixes: remove unused Node.js from test job, add concurrency groups and timeouts#20

Open
nightshift-micr wants to merge 1 commit intoMicrock:mainfrom
nightshift-micr:nightshift/ci-fixes
Open

[nightshift] ci-fixes: remove unused Node.js from test job, add concurrency groups and timeouts#20
nightshift-micr wants to merge 1 commit intoMicrock:mainfrom
nightshift-micr:nightshift/ci-fixes

Conversation

@nightshift-micr
Copy link
Copy Markdown

Automated by Nightshift v3 (GLM 5.1).

Task: ci-fixes
Category: pr
Repo: Microck/traccia

Changes

ci.yml

  • Remove unused Node.js setup from test job — the test job only runs Python lint/test, Node.js is only needed in the package job for npm smoke tests. This saves ~10-15s per matrix run (2 runs).
  • Add concurrency group with — prevents duplicate CI runs when pushing rapidly to the same branch.
  • Add timeout-minutes — test job: 15min, package job: 10min. Prevents hung workflows from burning runner minutes indefinitely.

release.yml

  • Add concurrency group with — prevents accidental double-publishing on race conditions while still grouping runs.
  • Add timeout-minutes to all jobs — build: 15min, publish jobs: 5min each.

Impact

  • ~30s faster CI per push (removed Node.js setup × 2 matrix runs)
  • Better resource management with concurrency control and timeouts
  • No behavior changes to test execution or build output

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