-
Notifications
You must be signed in to change notification settings - Fork 86
cargo-release setup #483
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
cargo-release setup #483
Conversation
03357aa to
64ea34c
Compare
schell
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I appreciate you putting this together @Firestar99 🙇
|
|
| cargo-run-wasm = "0.3.2" | ||
|
|
||
| [package.metadata.release] | ||
| release = false |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These shouldn't be needed as we already have publish = false?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
cargo release doesn't respect publish = false unfortunately
(which feels like an oversight on their part?)
Setup releases so that a simple
cargo release minor(orprerelease) in the workspace root should do it.Note that it's been quite difficult to test this configuration, like at all. Test releases (without
--execute) fail to verify the packaged crates due to using the "latest" version from crates.io instead of local changes. With the many breaking changes that requires upgrading more than one crate at once, basically all crates fail verification. Not even--no-verifyworks, since the generatedCargo.tomlwants a new feature from one of our crates, that doesn't exist onv0.9.0, so it won't even package it. Only when we actually do the full release, package verification should work, since we're actively publishing thev0.10.0(or*-alpha.0) crates and redownloading them. So for the next release, we just release immediately and hope nothing breaks 💀TLDR: Only the next release will show whether everything will just work or not, so I'll take the responsibility to do the next release and fix any issues as I encounter them.