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

Fix typos in README.md #5

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

bim9262
Copy link

@bim9262 bim9262 commented Mar 9, 2022

Found a few typos in the readme that tripped me up. Great tool by the way!! Thanks :)

@@ -46,11 +46,11 @@ allow = [

After that you can run:

#### `$ cargo lint fmt`

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This isn't a type unless you change the name of the command, and if you do, lots of things used by other people will break.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not sure what you mean, when I install it, it is found/valid only as cargo lints not lint

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@appetrosyan I didn't change anything to invoke the tool as lints vs lint.

 > cargo lint
error: no such subcommand: `lint`

        Did you mean `lints`?   
 > cargo lints
cargo-lints-lints 0.1.0

USAGE:
    cargo-lints lints [OPTIONS] <SUBCOMMAND>

FLAGS:
    -h, --help       Prints help information
    -V, --version    Prints version information

OPTIONS:
    -f, --file <file>    Path to lints.toml file

SUBCOMMANDS:
    fmt     Formats lints.toml file
    help    Prints this message or the help of the given subcommand(s)

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

Successfully merging this pull request may close these issues.

3 participants