|
4 | 4 | 2. Create your feature branch: `git checkout -b my-new-feature`
|
5 | 5 | 3. Test it: `cargo test`
|
6 | 6 | 4. Lint it: `cargo +beta clippy --all --all-targets -- -D warnings`
|
| 7 | + |
7 | 8 | > We use `cargo clippy` to ensure high-quality code and to enforce a set of best practices for Rust programming. However, not all lints provided by `cargo clippy` are relevant or applicable to our project.
|
8 | 9 | > We may choose to ignore some lints if they are unstable, experimental, or specific to our project.
|
9 | 10 | > If you are unsure about a lint, please ask us in the [rustup Discord channel](https://discord.com/channels/442252698964721669/463480252723888159).
|
10 |
| -5. Commit your changes: `git commit -am 'Add some feature'` |
11 |
| -6. Push to the branch: `git push origin my-new-feature` |
12 |
| -7. Submit a pull request :D |
| 11 | +
|
| 12 | +1. Commit your changes: `git commit -am 'Add some feature'` |
| 13 | +2. Push to the branch: `git push origin my-new-feature` |
| 14 | +3. Submit a pull request :D |
13 | 15 |
|
14 | 16 | For developing on `rustup` itself, you may want to install into a temporary
|
15 | 17 | directory, with a series of commands similar to this:
|
@@ -72,7 +74,7 @@ clippy is a moving target that can make it hard to merge for little benefit.
|
72 | 74 |
|
73 | 75 | We do ask that contributors keep the clippy status clean themselves.
|
74 | 76 |
|
75 |
| -Minimally, run `cargo +nightly clippy --all --all-targets -- -D warnings` before |
| 77 | +Minimally, run `cargo +beta clippy --all --all-targets -- -D warnings` before |
76 | 78 | submitting code.
|
77 | 79 |
|
78 | 80 | If possible, adding `--all-features` to the command is useful, but will require
|
|
0 commit comments