Slipstream/auto_review
1
0
Fork 0
Code Issues 2 Pull requests Projects Releases 27 Packages Activity Actions

fix(release): replace release-plz prep automation #100

Merged
jwilger merged 1 commit from fix-release-prep-package-verify into main 2026-05-05 17:14:42 -07:00
Conversation 1 Commits 1 Files changed 7 +260 -108
jwilger commented 2026-05-05 17:06:07 -07:00
Owner
Copy link

Summary

  • Replace release-plz release preparation with local root-release tooling and Forgejo tea PR creation.
  • Select the release bump by trying cargo-semver-checks as patch, then minor, then major, and use conventional commits only for changelog text.
  • Keep publish scoped to release/v branches and root metadata changes for the Docker image release.

Verification

  • nix develop --command bash tests/release_tooling_test.sh
  • nix build .#checks.x86_64-linux.release-tooling
  • nix develop --command scripts/release plan --workspace . --release-type patch
  • nix develop --command cargo semver-checks --workspace --baseline-rev v0.1.0 --release-type patch
## Summary - Replace release-plz release preparation with local root-release tooling and Forgejo tea PR creation. - Select the release bump by trying cargo-semver-checks as patch, then minor, then major, and use conventional commits only for changelog text. - Keep publish scoped to release/v branches and root metadata changes for the Docker image release. ## Verification - nix develop --command bash tests/release_tooling_test.sh - nix build .#checks.x86_64-linux.release-tooling - nix develop --command scripts/release plan --workspace . --release-type patch - nix develop --command cargo semver-checks --workspace --baseline-rev v0.1.0 --release-type patch
fix(release): replace release-plz prep automation
All checks were successful
CI / Nix flake check (pull_request) Successful in 2m40s
Details
CI / auto_review semantic review (pull_request) Successful in 1s
Details
auto_review auto_review: 1 warning
Publish release / release-publish (pull_request) Has been skipped
Details
5aa1c98155
auto-review approved these changes 2026-05-05 17:09:46 -07:00
auto-review left a comment
Copy link

This PR replaces the release-plz tool with a custom release preparation process using local scripts and tea for PR creation. The changes appear well-structured, but ensure that new commands and error handling are robust. The removal of release-plz.toml should be verified to ensure all configurations are covered by the new process.

Walkthrough

  • .forgejo/workflows/release-prepare.yml
    • Replaced release-plz with local scripts and tea for release preparation and PR creation.
    • Added logic to determine the release type using cargo semver-checks.
    • Ensure error handling for tea commands is robust.
  • .forgejo/workflows/release-publish.yml
    • Updated branch matching logic to align with new release branch naming.
  • release-plz.toml
    • Removed, indicating a shift away from release-plz. Verify all configurations are now handled by the new process.
  • scripts/release
    • Added a plan command to determine release version and type.
    • Ensure the new logic is thoroughly tested and documented.
This PR replaces the `release-plz` tool with a custom release preparation process using local scripts and `tea` for PR creation. The changes appear well-structured, but ensure that new commands and error handling are robust. The removal of `release-plz.toml` should be verified to ensure all configurations are covered by the new process. ## Walkthrough - **.forgejo/workflows/release-prepare.yml** - Replaced `release-plz` with local scripts and `tea` for release preparation and PR creation. - Added logic to determine the release type using `cargo semver-checks`. - Ensure error handling for `tea` commands is robust. - **.forgejo/workflows/release-publish.yml** - Updated branch matching logic to align with new release branch naming. - **release-plz.toml** - Removed, indicating a shift away from `release-plz`. Verify all configurations are now handled by the new process. - **scripts/release** - Added a `plan` command to determine release version and type. - Ensure the new logic is thoroughly tested and documented.
release-plz.toml
auto-review commented 2026-05-05 17:09:46 -07:00
Owner
Copy link

🟡 Warning: The removal of release-plz.toml indicates a shift away from using release-plz. Ensure that all necessary configurations are now handled by the new release process to avoid any disruptions.

🟡 **Warning:** The removal of `release-plz.toml` indicates a shift away from using `release-plz`. Ensure that all necessary configurations are now handled by the new release process to avoid any disruptions.
jwilger marked this conversation as resolved
Sign in to join this conversation.
Reviewers
No reviewers
auto-review
Labels
Clear labels   
bug

   
discussion

   
enhancement
  
bug
Something is not working

  
duplicate
This issue or pull request already exists

  
enhancement
New feature

  
help wanted
Need some help

  
invalid
Something is wrong

  
question
More information is needed

  
wontfix
This won't be fixed

No labels
bug
discussion
enhancement
bug
duplicate
enhancement
help wanted
invalid
question
wontfix
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
Slipstream/auto_review!100
No description provided.