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.

Sign up for GitHub

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

Jump to bottom

Setup rustfmt #80

Closed
iliekturtles opened this issue Apr 22, 2018 · 0 comments
Closed

Setup rustfmt #80

iliekturtles opened this issue Apr 22, 2018 · 0 comments

Comments

@iliekturtles
Copy link
Owner

iliekturtles commented Apr 22, 2018 •
edited
Loading

Currently uom tries to use most rustfmt styles options but doesn't actually use rustfmt because rustfmt overrides a couple manual formatting choices and there is no way to skip these currently (the #[cfg(...)]compile_error!(...); lines in src/lib.rs mostly).

  • rustfmt_skip doesn't seem to work.
  • file-lines doesn't have a negative option to only exclude certain lines.
@Aehmlo Aehmlo mentioned this issue May 1, 2018
Closed
iliekturtles added a commit that referenced this issue May 29, 2018
@iliekturtles
Setup rustfmt.
4441c92 
Included .rustfmt.toml allows `cargo fmt` to be run on `uom`.
Additionally the Travis CI build includes an additional job to validate
that any changes do not break formatting conventions. Resolves #80.
iliekturtles added a commit that referenced this issue May 29, 2018
@iliekturtles
Setup rustfmt.
3e09747 
Included .rustfmt.toml allows `cargo fmt` to be run on `uom`.
Additionally the Travis CI build includes an additional job to validate
that any changes do not break formatting conventions. Resolves #80.
iliekturtles added a commit that referenced this issue May 29, 2018
@iliekturtles
Setup rustfmt.
80ef52c 
Included .rustfmt.toml allows `cargo fmt` to be run on `uom`.
Additionally the Travis CI build includes an additional job to validate
that any changes do not break formatting conventions. Resolves #80.
iliekturtles added a commit that referenced this issue May 29, 2018
@iliekturtles
Setup rustfmt.
8d530ab 
Included .rustfmt.toml allows `cargo fmt` to be run on `uom`. `rustfmt`
0.4.2-stable or newer is required (distributed with `rustc` 1.26.1).
Additionally the Travis CI build includes an additional job to validate
that any changes do not break formatting conventions. Resolves #80.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Basic rustfmt setup
1 participant
@iliekturtles