Closed
Bug 1551078
Opened 5 years ago
Closed 5 years ago
Add support of rustfmt in mozlint
Categories
(Developer Infrastructure :: Lint and Formatting, task)
Developer Infrastructure
Lint and Formatting
Tracking
(firefox69 fixed)
RESOLVED
FIXED
mozilla69
| Tracking | Status | |
|---|---|---|
| firefox69 | --- | fixed |
People
(Reporter: Sylvestre, Assigned: Sylvestre)
References
(Blocks 2 open bugs, Regressed 1 open bug)
Details
Attachments
(1 file)
I think we should extend mozlint instead of creating yet a new infra (bug 1369792).
Two use cases.
Show the errors:
$ ./mach lint -l rustfmt js/rust/src/rust.rs
Update the code:
$ ./mach lint -l rustfmt js/rust/src/rust.rs --fix
To use it:
$ rustup component add rustfmt
$ export PATH=$PATH:~/.rustup/toolchains/stable-x86_64-unknown-linux-gnu/bin/
(will be improved later)
This could be copied for black and prettier
|
Assignee | |
Comment 1•5 years ago
|
||
Two use cases:
-
Show the errors
$ ./mach lint -l rustfmt js/rust/src/rust.rs -
Update the code
$ ./mach lint -l rustfmt js/rust/src/rust.rs --fix
Pushed by sledru@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/06fd76912201 Add rustfmt as part of mozlint r=ahal
|
||
Comment 3•5 years ago
|
||
| bugherder | ||
Status: NEW → RESOLVED
Closed: 5 years ago
status-firefox69:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla69
|
||
Updated•2 years ago
|
Product: Firefox Build System → Developer Infrastructure
You need to log in
before you can comment on or make changes to this bug.
Description
•