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

Rustfmt notravis #275

Closed
wants to merge 5 commits into from
Closed

Rustfmt notravis #275

wants to merge 5 commits into from

Conversation

Labels
None yet
Projects
None yet
3 participants
@teor2345
Copy link
Contributor

@teor2345 teor2345 commented Aug 16, 2018

From 'rustfmt-notravis' at ​https://gitgud.io/onionk/tor.git

cypherpunks added 4 commits Aug 11, 2018
Fix typo from fe66d06.
The exclamation point is what lets an attribute apply to
an entire crate, without the ! it's practically a placebo.

Fix on commits af182d4 and
b605929, and note there are
still missing docs in both crypto and protover, for now.

https://doc.rust-lang.org/reference/attributes.html
These are the 12 stable and documented configuration options,
set to their default values.

use_small_heuristics is only stabilized in rustfmt 0.9, so maintain
support for 0.8.x for now by commenting it out.

comment_width is unstable and did nothing, since wrap_comments defaults
to false.

Default values gotten from `rustfmt --print-config default rustfmt.toml`.

https://github.com/rust-lang-nursery/rustfmt/blob/e7932fa9c2591c45a37a24305de90cb63128afcf/Configurations.md
Copy link
Contributor Author

@teor2345 teor2345 left a comment

Please fix the documentation typo in your changes to 132c795.

@@ -104,7 +104,7 @@ repo.
Documentation
---------------

You MUST include `#[deny(missing_docs)]` in your crate.
You MUST include `#[!deny(missing_docs)]` in your crate.
Copy link
Contributor Author

@teor2345 teor2345 Aug 16, 2018

Typo: #![deny(missing_docs)]

@coveralls
Copy link

@coveralls coveralls commented Aug 16, 2018

Coverage Status

Coverage increased (+0.02%) to 59.546% when pulling da6054f on teor2345:rustfmt-notravis into 34083f0 on torproject:master.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment