Skip to content

chore(email): leave the version bump to Release Control - #980

Merged
rohitg00 merged 1 commit into
mainfrom
fix/email-version-owned-by-release-control
Aug 28, 2026
Merged

chore(email): leave the version bump to Release Control#980
rohitg00 merged 1 commit into
mainfrom
fix/email-version-owned-by-release-control

Conversation

@rohitg00

@rohitg00 rohitg00 commented Aug 28, 2026

Copy link
Copy Markdown
Contributor

PR #979 raised email/Cargo.toml from 0.1.7 to 0.2.0 by hand. Release Control owns worker versions: it derives the candidate from the registry base (0.1.7) and commits the prepare vX bump itself, so the hand bump made it plan 0.2.1-rc.1 and its prepare step failed at dispatch (prepare_release:email: dispatch outcome is unknown), leaving the worker:email lock held.

This puts the manifest back at 0.1.7 (Cargo.toml + Cargo.lock only). The intended release stays 0.2.0: pick the minor stable goal in Release Control and it will derive 0.2.0-rc.1. The README already documents 0.2.0.

Summary by CodeRabbit

  • Chores
    • Updated the email package version to 0.1.7.

PR #979 raised email/Cargo.toml to 0.2.0 by hand. Release Control owns worker versions: it derives the candidate from the registry base and commits the prepare bump itself, and the hand bump made it plan 0.2.1-rc.1 against a 0.1.7 base, which its prepare step rejected. Back to 0.1.7 so the next release intent (minor) yields 0.2.0-rc.1.
@rohitg00 rohitg00 added the no-ticket PR deliberately has no Linear ticket (bump/typo/CI-only) label Aug 28, 2026
@vercel

vercel Bot commented Aug 28, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
workers Ready Ready Preview Aug 28, 2026 1:38pm
workers-tech-spec Ready Ready Preview Aug 28, 2026 1:38pm

Request Review

@github-actions

Copy link
Copy Markdown
Contributor

skill-check — worker

0 verified, 69 skipped (no docs/).

Layer Result
structure
vale
ai
render

Four for four. Nicely done.

@rohitg00
rohitg00 merged commit 281ff0c into main Aug 28, 2026
18 of 21 checks passed
@rohitg00
rohitg00 deleted the fix/email-version-owned-by-release-control branch August 28, 2026 13:38
@coderabbitai

coderabbitai Bot commented Aug 28, 2026

Copy link
Copy Markdown

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 4db227e0-c779-4a28-97de-a3eb2325375d

📥 Commits

Reviewing files that changed from the base of the PR and between 9aba119 and d440a52.

⛔ Files ignored due to path filters (1)
  • email/Cargo.lock is excluded by !**/*.lock
📒 Files selected for processing (1)
  • email/Cargo.toml

📝 Walkthrough

Walkthrough

The email package version changes from 0.2.0 to 0.1.7 in its Cargo manifest.

Changes

Email package version

Layer / File(s) Summary
Update package version
email/Cargo.toml
The package version changes from 0.2.0 to 0.1.7.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Suggested reviewers: guibeira

Poem

A rabbit checks the version line
A smaller number now will shine
The manifest rests, neat and bright
One tiny change completes the night
Ears up high, the build takes flight

✨ Finishing Touches 💡 1
🛠️ Fix failing CI checks 💡
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/email-version-owned-by-release-control

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-ticket PR deliberately has no Linear ticket (bump/typo/CI-only)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant