-
Notifications
You must be signed in to change notification settings - Fork 95
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
Don't panic from WorkspaceDefaultMembers? #254
Comments
As a workaround, you can use |
You can use the https://crates.io/crates/rustc_version crate to check whether your rust is new enough for this field |
Also, just to confirm, this hinges on the version of |
yes |
I ran into this issue also. Would it be acceptable to add a method to the |
I'm also interested in this for |
From cargo-msrv there would also be interest, as we work a lot with older cargo releases. Happy to help if necessary. |
I guess. User needs trump whatever reasons I have |
I opened a PR in #282. However, if you feel that this is not in scope for this project, then that's of course also fine. |
Hi, https://docs.rs/cargo_metadata/0.18.1/cargo_metadata/struct.WorkspaceDefaultMembers.html documents that it will panic on cargo <1.70, and it does.
For cargo-mutants, I'd like to use this but also support older versions of cargo, so it would be really nice if this was a runtime error rather than a panic. Or at least, if there was a documented version field that I could check before looking at it. Possibly I am missing some subtlety about why that's not feasible.
Thanks for a really helpful crate.
The text was updated successfully, but these errors were encountered: