Closed
Bug 1551078
Opened 6 years ago
Closed 6 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)
|
(deleted),
text/x-phabricator-request
|
Details |
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•6 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•6 years ago
|
||
| bugherder | ||
Status: NEW → RESOLVED
Closed: 6 years ago
status-firefox69:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla69
|
||
Updated•6 years ago
|
|
||
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
•