Closed
Bug 1657604
Opened 4 years ago
Closed 4 years ago
Support multiple linter per yaml file
Categories
(Developer Infrastructure :: Lint and Formatting, enhancement, P3)
Developer Infrastructure
Lint and Formatting
Tracking
(firefox81 fixed)
RESOLVED
FIXED
81 Branch
| Tracking | Status | |
|---|---|---|
| firefox81 | --- | fixed |
People
(Reporter: ahal, Assigned: ahal)
Details
Attachments
(1 file)
Currently we can only define one linter per yaml file. But in certain cases (like the rejected-words linter), it'll be more convenient to define a bunch in a single file.
Yaml supports "documents" which are delimited by ---. It should be pretty easy to parse them by changing this call to safe_load_all and looping over the generated documents.
|
Assignee | |
Comment 1•4 years ago
|
||
We should also support both the yaml file name, as well as the linter name in the --linter argument (where passing the file name now runs all linters contained therein).
|
|
Assignee | |
Comment 2•4 years ago
|
||
|
||
Updated•4 years ago
|
Assignee: nobody → ahal
Status: NEW → ASSIGNED
Pushed by sledru@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/251f6148bdb8 [mozlint] Support multiple linters per yaml file, r=linter-reviewers,sylvestre
|
||
Comment 4•4 years ago
|
||
| bugherder | ||
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
status-firefox81:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 81 Branch
|
||
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
•