Closed Bug 1657604 Opened 6 months ago Closed 6 months ago

Support multiple linter per yaml file

Categories

(Firefox Build System :: Lint and Formatting, enhancement, P3)

enhancement

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.

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: 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
Status: ASSIGNED → RESOLVED
Closed: 6 months ago
Resolution: --- → FIXED
Target Milestone: --- → 81 Branch
You need to log in before you can comment on or make changes to this bug.