Skip to content

ci: Fix bump-api-schema workflow to merge PRs immediately#16827

Merged
sergical merged 1 commit intomasterfrom
fix/bump-api-schema-merge-immediately
Mar 6, 2026
Merged

ci: Fix bump-api-schema workflow to merge PRs immediately#16827
sergical merged 1 commit intomasterfrom
fix/bump-api-schema-merge-immediately

Conversation

@sergical
Copy link
Member

@sergical sergical commented Mar 6, 2026

DESCRIBE YOUR PR

The bump-api-schema-sha.yml workflow creates PRs with --auto merge, but checks never pass — causing a pile-up of open "Bump API schema" PRs. Since getsentry bot has been added to bypass PR requirements, we can use --admin to merge immediately.

Changes:

  • Add no-op guard after sed to skip when the SHA is unchanged (prevents duplicate PRs)
  • Switch gh pr merge --squash --autogh pr merge --squash --admin (bypass branch protections, merge immediately)

IS YOUR CHANGE URGENT?

  • Urgent deadline (GA date, etc.):
  • Other deadline:
  • None: Not urgent, can wait up to 1 week+

SLA

  • Teamwork makes the dream work, so please add a reviewer to your PRs.
  • Please give the docs team up to 1 week to review your PR unless you've added an urgent due date to it.
    Thanks in advance for your help!

PRE-MERGE CHECKLIST

  • Checked Vercel preview for correctness, including links
  • PR was reviewed and approved by any necessary SMEs (subject matter experts)
  • PR was reviewed and approved by a member of the Sentry docs team

- Add no-op guard to skip when SHA is unchanged, preventing duplicate PRs
- Switch from --auto to --admin merge to bypass branch protections,
  since checks never pass causing a pile-up of open PRs

Co-Authored-By: Claude <noreply@anthropic.com>
@vercel
Copy link

vercel bot commented Mar 6, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
develop-docs Ready Ready Preview, Comment Mar 6, 2026 6:40pm
sentry-docs Ready Ready Preview, Comment Mar 6, 2026 6:40pm

Request Review

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