Skip to content

feat: verify branch protection before auto-merge - #20

Merged
HugoXK merged 2 commits into
mainfrom
feat/branch-protection
Nov 4, 2026
Merged

HugoXK merged 2 commits into
mainfrom
feat/branch-protection

Conversation

@HugoXK

@HugoXK HugoXK commented Nov 4, 2026

Copy link
Copy Markdown
Owner

Written-by: HugoXK

Verifies branch protection before auto-merge.

  • Read the target branch protection rule.
  • Verify required status checks and required approvals.
  • Support squash, merge and rebase.
  • Refuse auto-merge when protection cannot be read or a required gate is missing.
  • Expose the protection state in repository settings.

Ran:

  • npm run check — pass (104 tests, build)
  • node scripts/changelog.mjs check — pass
    Not run: production deploy.

Read the branch protection rule, verify required status checks and approvals, support squash/merge/rebase, expose protection state in settings, and block auto-merge when protection is unavailable.
Comment thread src/app.ts Fixed
Comment thread src/app.ts Fixed
Read the branch protection result once so CodeQL no longer reports unused initial values.
@HugoXK

HugoXK commented Nov 4, 2026

Copy link
Copy Markdown
Owner Author

Verdict: PASS at f34a585
Written-by: Reviewer
Posted-by: HugoXK

Ran:

  • npm run check — pass (104 tests, build)
  • node scripts/changelog.mjs check — pass
    Not run: production deploy

Addressed the CodeQL useless-assignment findings by reading the protection result once.

@HugoXK
HugoXK merged commit 2b3ee83 into main Nov 4, 2026
14 of 15 checks passed
@HugoXK
HugoXK deleted the feat/branch-protection branch November 4, 2026 08:00
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