-
Notifications
You must be signed in to change notification settings - Fork 22
feat: generalizing the command line #58
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: master
Are you sure you want to change the base?
Conversation
@frewsxcv This PR is ready for review. |
Also pinging @Manishearth, there's been a couple attempts at adding this (see #56, #48, and this), it would be nice to finally see this get added and a new release out. I'd love to be able to migrate our projects to cargo-nextest as well, but this is a blocker, and cargo-hack doesn't have nearly as good customization as this (see the Cargo.toml section), otherwise I'd use that. |
This is a big change, and I'm pretty busy these days. I agree that this would be a good feature to add. I'll perform a partial review but I really can't guarantee anything. |
This PR provides using
cargo-all-features
with any other cargo subcommands and respect and pass arguments to them.The new command will look like this:
cargo all-features [subcommand] [subcommand args] [--] [args passed to subcommand with '--']
Example:
Note: All three pre-defined commands (
cargo-[test,check,build]-all-features
) have been removed and replaced with a new generic one. If you are willing to keep backward compatibility on this, please let me know to add them.