Skip to content

fix: make cargo package versions explicit for release-please#10

Merged
anandpant merged 1 commit into
mainfrom
fix/release-please-cargo-version
Apr 20, 2026
Merged

fix: make cargo package versions explicit for release-please#10
anandpant merged 1 commit into
mainfrom
fix/release-please-cargo-version

Conversation

@anandpant

@anandpant anandpant commented Apr 20, 2026

Copy link
Copy Markdown
Contributor

Summary

  • make each Cargo package manifest use an explicit version string
  • unblock release-please on main

Why

The merged main commit 494d7ea left normal CI healthy, but release / 01-release-please failed with:

  • package manifest at apps/convex-sync/Cargo.toml has an invalid [package.version]

The failure was caused by version.workspace = true in the package manifests. The cargo-workspace plugin expects explicit package versions.

Changes

  • set version = "0.0.1" in:
    • apps/convex-sync/Cargo.toml
    • crates/convex-cdc-core/Cargo.toml
    • crates/convex-target-s3/Cargo.toml

Verification

  • cargo fmt --all --check
  • cargo test --workspace
  • cargo run -p convex-sync -- --version
  • actionlint -config-file .github/actionlint.yaml .github/workflows/*.yml .depot/workflows/*.yml

Open in Devin Review

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 1 additional finding.

Open in Devin Review

@anandpant
anandpant merged commit 00f3993 into main Apr 20, 2026
7 checks passed
@anandpant
anandpant deleted the fix/release-please-cargo-version branch April 20, 2026 06:56
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