diff mbox series

[1/3] CONTRIBUTING.md: Update references, add rustfmt information

Message ID 20180723032129.4365-1-andrew.donnellan@au1.ibm.com
State Superseded
Headers show
Series [1/3] CONTRIBUTING.md: Update references, add rustfmt information | expand

Commit Message

Andrew Donnellan July 23, 2018, 3:21 a.m. UTC
The Rust Guidelines have disappeared, so replace that with a link to the
current Rust Style Guide.

Document the new rustfmt git pre-commit hook.

Update the link to clippy and get rid of outdated installation information
that's better documented in the clippy repo anyway.

Signed-off-by: Andrew Donnellan <andrew.donnellan@au1.ibm.com>
 CONTRIBUTING.md | 17 +++++++++--------
 1 file changed, 9 insertions(+), 8 deletions(-)
diff mbox series


index 8d040725f3ca..a6daa1feff25 100644
@@ -24,14 +24,15 @@  relevant. Specifically, pay attention to:
  * Separating logical changes into separate patches
  * Plain-text patches, as generated by `git send-email`
-We like to follow the [Rust Guidelines](https://aturon.github.io/)
-where possible - patches to fix existing non-compliant code are
-If you're using a nightly Rust toolchain, you can use the
-[clippy](https://github.com/Manishearth/rust-clippy) linter: `cargo
-+nightly install clippy` to install, and `cargo +nightly clippy` to
+We like to follow the [Rust Style
+wherever possible - patches to fix any non-compliant code are welcome!
+The `setup_rustfmt_hook.sh` script in the root of the repository will
+set up a local pre-commit hook that runs `rustfmt` over your changes -
+we strongly recommend using this.
+We also recommend using the
+[clippy](https://github.com/rust-lang-nursery/rust-clippy) linter.
 When your patch involves creating a new file, where possible please
 use a header along the lines of: