Skip to content

ci: do not block Changesets version on snapshot publish - #866

Merged
patroza merged 1 commit into
mainfrom
t3code/version-independent-snapshot
Aug 14, 2026
Merged

ci: do not block Changesets version on snapshot publish#866
patroza merged 1 commit into
mainfrom
t3code/version-independent-snapshot

Conversation

@patroza

@patroza patroza commented Aug 14, 2026

Copy link
Copy Markdown
Member

Summary

On main, Changesets calls the reusable CI workflow and then runs version with needs: [ci]. Publish Snapshot lived inside that reusable workflow, so version waited on pkg.pr.new even though the two jobs do not depend on each other.

Snapshot is now its own reusable workflow:

  • PRs / workflow_dispatch: still publish after build (lint/test stay parallel).
  • main (release.yaml): snapshot and version both needs: [ci], so they start together after lint/build/test.

Version still waits for the quality gates. It no longer waits for snapshot success or failure.

Test plan

  • PR CI still runs Lint, Build, Test, and Publish Snapshot after build
  • On the next main push (non-version-bump, non-bot), Changesets shows snapshot and version as siblings after ci
  • A snapshot failure must not prevent version from starting
  • Version-bump commits (chore: update versions) still skip snapshot

View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need. Autofix is disabled.

The reusable CI workflow included Publish Snapshot, so the version job
waited on pkg.pr.new. Snapshot now lives in its own reusable workflow:
PRs still publish after build, and on main snapshot is a sibling of
version after lint/build/test.
@patroza
patroza marked this pull request as ready for review August 14, 2026 06:29
@pkg-pr-new

pkg-pr-new Bot commented Aug 14, 2026

Copy link
Copy Markdown

Open in StackBlitz

@effect-app/cli

npm i https://pkg.pr.new/effect-app/libs/@effect-app/cli@866

effect-app

npm i https://pkg.pr.new/effect-app/libs/effect-app@866

@effect-app/eslint-codegen-model

npm i https://pkg.pr.new/effect-app/libs/@effect-app/eslint-codegen-model@866

@effect-app/eslint-shared-config

npm i https://pkg.pr.new/effect-app/libs/@effect-app/eslint-shared-config@866

@effect-app/infra

npm i https://pkg.pr.new/effect-app/libs/@effect-app/infra@866

@effect-app/vue

npm i https://pkg.pr.new/effect-app/libs/@effect-app/vue@866

@effect-app/vue-components

npm i https://pkg.pr.new/effect-app/libs/@effect-app/vue-components@866

commit: 3df05e2

@patroza
patroza merged commit ec25d35 into main Aug 14, 2026
6 checks passed
@patroza
patroza deleted the t3code/version-independent-snapshot branch August 14, 2026 06:35
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