Skip to content
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

Suggestion: Rename --all to --workspace #4441

Closed
behnam opened this issue Sep 27, 2020 · 1 comment
Closed

Suggestion: Rename --all to --workspace #4441

behnam opened this issue Sep 27, 2020 · 1 comment

Comments

@behnam
Copy link

behnam commented Sep 27, 2020

In rust-lang/cargo#6977, cargo's main subcommands, including check, build, and test, were updated to use --workspace instead of --all, to prevent the confusion about what the phrase "all" refers to.

Looks like cargo fmt still uses --all and have not switched to --workspace. Suggesting making that transition here, as well.

What do you think?

@behnam behnam added the bug Panic, non-idempotency, invalid code, etc. label Sep 27, 2020
@calebcartwright calebcartwright removed the bug Panic, non-idempotency, invalid code, etc. label Sep 27, 2020
@calebcartwright
Copy link
Member

Thanks for reaching out @behnam

The --all flag means something different in the cargo fmt context, more info here, and this is also a duplicate of #3911 so I'm going to close.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants