Skip to content

ci: adopt shared @btravstack reusable CI + release workflows#594

Merged
btravers merged 3 commits into
mainfrom
ci/use-reusable-workflows
Jul 24, 2026
Merged

ci: adopt shared @btravstack reusable CI + release workflows#594
btravers merged 3 commits into
mainfrom
ci/use-reusable-workflows

Conversation

@btravers

Copy link
Copy Markdown
Contributor

What

Migrate CI and release to the shared reusable workflows in btravstack/config (the proven pattern, already green in unthrown).

CI (.github/workflows/ci.yml)

  • Replaced the common jobs (format, lint, typecheck, knip, test, build, security-audit, changeset) with a single call to btravstack/config/.github/workflows/ci-reusable.yml@main.
    • changeset: true — the changeset gate now runs inside the reusable ci job (and is auto-skipped for the changeset-release/main PR).
    • test-command: "pnpm test -- --coverage --reporter=default --reporter=github-actions" — preserves the non-default test command.
    • knip left at its default (true) — the old knip job was plain (pnpm exec knip ...), no extra steps.
    • security-audit left at its default (true).
  • Kept the bespoke jobs verbatim: test-integration, bundle-size (needs: build -> needs: ci), package-check, node-floor.

Release (.github/workflows/release.yml)

  • Replaced the inline changesets steps with a call to btravstack/config/.github/workflows/release-reusable.yml@main, passing RELEASE_PAT. The old version/publish commands (pnpm run version / pnpm run release) are the reusable's defaults, so no overrides needed.

Note

  • Dropped the unit-test coverage-artifact upload (coverage-report) — the reusable ci job runs tests but does not upload coverage. The integration coverage upload stays (it lives in the bespoke test-integration job).
  • deploy-docs.yml and .github/actions/setup untouched.

Copilot AI review requested due to automatic review settings July 24, 2026 08:17

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Migrates this repository’s CI and release pipelines to the shared reusable workflows in btravstack/config, consolidating the common “gate” steps into a single reusable job while keeping repo-specific jobs in-repo.

Changes:

  • Replaced the multi-job CI gate (format/lint/typecheck/knip/test/build/audit/changeset) with a single call to btravstack/config/.github/workflows/ci-reusable.yml@main.
  • Replaced the inline Changesets-based release workflow with a call to btravstack/config/.github/workflows/release-reusable.yml@main, passing RELEASE_PAT.
  • Updated the bundle-size job dependency to wait for the new ci gate job.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
.github/workflows/ci.yml Collapses the common CI gate into a reusable workflow call; keeps bespoke jobs (integration, bundle-size, package-check, node-floor).
.github/workflows/release.yml Delegates release/version/publish to the shared reusable release workflow using RELEASE_PAT.

Comment thread .github/workflows/release.yml
Comment thread .github/workflows/ci.yml Outdated
Comment thread .github/workflows/ci.yml Outdated
Comment thread .github/workflows/ci.yml Outdated
@btravers
btravers merged commit ba97361 into main Jul 24, 2026
12 checks passed
@btravers
btravers deleted the ci/use-reusable-workflows branch July 24, 2026 09:55
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