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.

Sign up for GitHub

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

Jump to bottom

fix(help):Clarify install's positional #12841

Merged
merged 2 commits into from
Oct 18, 2023
Merged

fix(help):Clarify install's positional #12841

merged 2 commits into from
Oct 18, 2023

Commits on Oct 17, 2023

  1. fix(help): Consistently use SCREAMING_CASE for value names 

    - We have some positional value names that are SCREAMING_CASE and some
  that aren't
- All of the value names for our flags are already SCREAMING_CASE
    @epage
    epage committed Oct 17, 2023
    Configuration menu
    Copy the full SHA
    99f917a View commit details
    Browse the repository at this point in the history

  2. fix(help): Clarify install's positional 

    - That a version is accepted
- That you are selecting from the source a package which led to part of
  the confusion in rust-lang#4830

I wonder if we should rename our `CRATE` value names to `PKG`/`PACKAGE`
    @epage
    epage committed Oct 17, 2023
    Configuration menu
    Copy the full SHA
    5f05969 View commit details
    Browse the repository at this point in the history