-
Notifications
You must be signed in to change notification settings - Fork 31
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
cargo msrv find --set #1047
Comments
I would welcome an alias, because similarity in naming between similar options does matter for ease of use. See https://github.com/foresterre/cargo-msrv/blob/main/src/cli/mod.rs#L168 |
Oh wow, how did I miss that :-) Thank you! I'll submit a PR for an alias then. |
cdown
added a commit
to cdown/cargo-msrv
that referenced
this issue
Nov 5, 2024
Introduce --set as an alias for the --write-msrv option in `cargo msrv find' to maintain consistency with `cargo msrv set'. Closes foresterre#1047.
cdown
added a commit
to cdown/cargo-msrv
that referenced
this issue
Nov 5, 2024
Introduce --set as an alias for the --write-msrv option in `cargo msrv find` to maintain consistency with `cargo msrv set`. Closes foresterre#1047.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
One of my usual workflows during dep upgrades is to do
cargo msrv find
, thencargo msrv set
. But this is a pain, because it requires manually copying the new version tocargo msrv set
. My workflow would benefit from something likecargo msrv find --set
, which would set the found MSRV.This looks relatively straightforward to implement, but before I submit a PR I wanted to check on the ergonomics and concept. Thanks!
The text was updated successfully, but these errors were encountered: