Skip to content

Fix sync-version-to-dev: create PR instead of direct push#57

Merged
flyon merged 1 commit intodevfrom
fix/sync-version-to-dev-via-pr
Apr 3, 2026
Merged

Fix sync-version-to-dev: create PR instead of direct push#57
flyon merged 1 commit intodevfrom
fix/sync-version-to-dev-via-pr

Conversation

@flyon
Copy link
Copy Markdown
Contributor

@flyon flyon commented Apr 3, 2026

Summary

The sync-version-to-dev CI job fails because the dev branch has branch protection rules requiring PRs. The job was trying to git push directly to dev.

Fix: Changed the job to create a branch (chore/sync-version-{version}) and open a PR to dev via gh pr create instead of pushing directly.

Changes

  • .github/workflows/publish.yml: sync-version-to-dev job now:
    • Creates a chore/sync-version-X.Y.Z branch
    • Commits the version bump there
    • Opens a PR to dev using gh pr create
    • Added pull-requests: write permission

Test plan

  • Workflow syntax is valid YAML
  • gh pr create uses GITHUB_TOKEN which is already available
  • Only runs when version actually differs (existing if check preserved)

https://claude.ai/code/session_019MDcpswuKKNx9YhC2S2vT7

The dev branch has branch protection requiring PRs. Changed the
sync-version-to-dev job to create a branch and open a PR via gh cli
instead of pushing directly to dev.

https://claude.ai/code/session_019MDcpswuKKNx9YhC2S2vT7
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 3, 2026

⚠️ No changeset found. If this PR should trigger a release, run npx changeset and commit the generated file.

If this change doesn't need a release (docs, CI config, etc.), you can ignore this message.

@flyon flyon merged commit daa7fc8 into dev Apr 3, 2026
4 checks passed
@flyon flyon deleted the fix/sync-version-to-dev-via-pr branch April 3, 2026 15: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