Skip to content

fix: publish release-plz releases after updating notes - #175

Merged
robjtede merged 2 commits into
mainfrom
chore/undraft-releases
Oct 9, 2026
Merged

robjtede merged 2 commits into
mainfrom
chore/undraft-releases

Conversation

@robjtede

@robjtede robjtede commented Oct 8, 2026 •

Copy link
Copy Markdown
Member

Summary

  • Create release-plz releases as drafts.
  • Update release notes from the changelog, then publish the releases with --then-undraft.
  • Update the x52dev/nix flake lock input so CI has the new flag.

Validation

  • Reviewed the workflow, release-plz config, and flake.lock diff.
  • Did not run project tests; these changes only affect release configuration.

Summary by CodeRabbit

  • Release Process
    • Release notes are updated before a draft release is published, keeping the notes current when a release becomes available.
    • Releases are prepared as drafts before publication.

@coderabbitai

coderabbitai Bot commented Oct 8, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

Warning

Review limit reached

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Next included review available in 31 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

Learn how review limits work.

Review configuration:

⚙️ Run configuration
  • Configuration used: Organization UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: c208f622-87dc-44bd-84a6-2cc220af31c2
📥 Commits

Reviewing files that changed from the base of the PR and between 1304a75 and b9c6d4d.

📒 Files selected for processing (1)
  • .github/workflows/ci.yml

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
  • Configuration used: Organization UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: a3982254-a283-43e6-b98c-0e7ec2f53b55
📥 Commits

Reviewing files that changed from the base of the PR and between b4c157a and 1304a75.

⛔ Files ignored due to path filters (1)
  • flake.lock is excluded by !**/*.lock
📒 Files selected for processing (2)
  • .github/workflows/release.yml
  • .release-plz.toml

Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.


📝 Walkthrough

Walkthrough

The workspace release configuration now creates draft releases. The release workflow runs x52-update-release-notes with the --then-undraft option.

Changes

Release draft handling

Layer / File(s) Summary
Draft release handling
.release-plz.toml, .github/workflows/release.yml
The workspace release configuration enables draft releases. The release-notes command adds --then-undraft.

Priority: ➖ Normal

Estimated code review effort: 2 (Simple) | ~5 minutes

Change: Bug fix

Merge Risk: ⚪ Minimal · up to 1304a

No specific release failure is established in the reviewed changes. The behavior of the locked notes command remains unverified, so confirm publication in normal release checks.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check Passed The title clearly and concisely describes the main change: publishing release-plz releases after updating their notes.
Docstring Coverage Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches 💡 1
🛠️ Fix failing CI checks 💡
  • Commit to this branch
  • Create a new PR
  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@robjtede
robjtede added this pull request to the merge queue Oct 9, 2026
Merged via the queue into main with commit 0b31fbd Oct 9, 2026
10 checks passed
@robjtede
robjtede deleted the chore/undraft-releases branch October 9, 2026 00:03
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