add publish job and dry-run to CI - #4
Conversation
This sounds good, but what exactly do you mean by "cutting a release" ? |
|
Haha fair question! Example workflow: |
|
Oh this is a great idea, yes please do add that! |
d12d981 to
6e04efa
Compare
|
Added the publish workflow as a separate file, Two things to set up before the first release:
The trigger is It runs |
|
...and just as I wanted to merge this, I encountered the very reason I need this: I have released v0.0.9 yesterday without pushing the commit from which I released, and have since merged your other PRs, which means In order to preserve the release commit I had to rebase origin/master onto my local master and force-push it, can you rebase your PR onto that please? Thanks for proactively making sure this won't happen again lol |
|
Oh and please do limit the trigger to version tags as you described while you're at it |
6e04efa to
3ff106a
Compare
|
Done and done :) |
|
tyvm! |
Adds a
cargo publish --dry-run --workspacejob. Should have put this in the first CI PR,I've found it very helpful.
--workspaceverifiesazoagainst the locally packagedazo-sysinstead of the versionon crates.io, so the two crates are checked as the coordinated release they actually are
I can also add a real publish job triggered when cutting a release, if you want that. I prefer
running publish from the repo, that way I know everything I publish is committed and pushed.