Open MarijnS95 opened 1 month ago
For completeness, I just tried without the commit that adds the publish
field to all our crates. In detached head mode, it's not happy:
$ cargo smart-release --registry that-registry --dry-run-cargo-publish
[WARN ] Consider running with --update-crates-index to assure bumping on demand uses the latest information
thread 'main' panicked at /home/marijn/.cargo/registry/src/index.crates.io-6f17d22bba15001f/cargo-smart-release-0.21.3/src/git/history.rs:40:40:
internal error: entered unreachable code: handled above
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
Checking out a branch to circumvent that error, the dry-run logs show up but unfortunately without any hint whether --registry
is picked up correctly and pushing to the right place.
Found what's happening: for some reason PublishDisabledInManifest
isn't printed nicely in the logs. But following the code trail:
This should check if the selected registry is set.
Custom registries aren't what I had in mind when writing this program, hence it's very unsupported.
New features can be added by the community, but as can be followed-up on in this PR the process for getting them merged is so special that it might be easier to look for other tools or combinations of tools.
@Byron thanks! I've already written up a patch but have yet to test it, was about to send you a pull request to get the ball rolling. Looks like we'll have to write some tests which is good, after first discussing what the right thing to do is. Let's leave that for the PR context.
Perhaps this --registry
cmdline argument isn't really playing ball with publish = ["specific-registry"]
and/or [registry] default = "..."
in .cargo/config.toml
, but cargo publish
has it as well so we can figure something out.
Hi! I've got a workspace setup where we use the
[package] publish
field to limit which registries that crate can be published to (an array of just one item).Unfortunately when running
cargo smart-release --dry-run-cargo-publish
or evencargo smart-release --registry that-registry --dry-run-cargo-publish
, the following error shows up:Is
cargo-smart-release
not understanding thatpublish
attribute? Let me know how I can continue to debug this :)