Skip to content

Commit

Permalink
Address review comments
Browse files Browse the repository at this point in the history
  • Loading branch information
flip1995 committed Jul 24, 2020
1 parent 17903f6 commit 3a4cc9f
Showing 1 changed file with 6 additions and 3 deletions.
9 changes: 6 additions & 3 deletions doc/basics.md
Expand Up @@ -48,6 +48,7 @@ this toolchain, you can just use the `setup-toolchain.sh` script or use
sh setup-toolchain.sh
# OR
cargo install rustup-toolchain-install-master
# For better IDE integration also add `-c rustfmt -c rust-src` (optional)
rustup-toolchain-install-master -f -n master -c rustc-dev -c llvm-tools
rustup override set master
```
Expand All @@ -62,8 +63,8 @@ cargo build # builds Clippy
cargo test # tests Clippy
```

Since Clippys test suite is pretty big, there are some commands that only run a
subset of Clippys tests:
Since Clippy's test suite is pretty big, there are some commands that only run a
subset of Clippy's tests:

```bash
# only run UI tests
Expand All @@ -74,7 +75,7 @@ TESTNAME="test_" cargo uitest
cargo test --test dogfood
```

If the output of a UI test differs from the expected output, you can update the
If the output of a [UI test] differs from the expected output, you can update the
reference file with:

```bash
Expand All @@ -87,6 +88,8 @@ or if you modify a test file to add a test case.
_Note:_ This command may update more files than you intended. In that case only
commit the files you wanted to update.

[UI test]: https://rustc-dev-guide.rust-lang.org/tests/adding.html#guide-to-the-ui-tests

## `cargo dev`

Clippy has some dev tools to make working on Clippy more convenient. These tools
Expand Down

0 comments on commit 3a4cc9f

Please sign in to comment.