Skip to content

fix(ci): merge release commits through prs - #132

Merged
hadrien merged 1 commit into
mainfrom
fix/release-via-pr
Aug 2, 2026
Merged

fix(ci): merge release commits through prs#132
hadrien merged 1 commit into
mainfrom
fix/release-via-pr

Conversation

@hadrien

@hadrien hadrien commented Aug 2, 2026

Copy link
Copy Markdown
Owner

Problem

  • Protected main rejects direct release pushes after the exact SHA passes every required check.
  • The ruleset reports those successful checks as expected during direct promotion.

Solution

  • Open a conventional PR for each generated release commit.
  • Dispatch and wait for CI on the PR head.
  • Squash-merge through the protected-main API with a pinned SHA.
  • Tag the resulting main commit before publishing.
  • Preserve docs, GitHub release notes, and PyPI publication.
  • Keep every main mutation inside the normal ruleset path.

fs-3z4

@codecov

codecov Bot commented Aug 2, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 97.74%. Comparing base (5dd5369) to head (b9f5f8a).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #132   +/-   ##
=======================================
  Coverage   97.74%   97.74%           
=======================================
  Files           1        1           
  Lines         133      133           
  Branches        7        7           
=======================================
  Hits          130      130           
  Misses          3        3           
Flag Coverage Δ
fastapi-post121 97.74% <ø> (ø)
fastapi-pre121 97.74% <ø> (ø)
sqlmodel 97.74% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

@hadrien
hadrien merged commit 2b78467 into main Aug 2, 2026
7 checks passed
@hadrien
hadrien deleted the fix/release-via-pr branch August 2, 2026 14:17
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