Closed
Bug 1808288
Opened 2 years ago
Closed 2 years ago
file-whitespace/file-perm linters won't be run on patches that only touch idl/webidl files
Categories
(Developer Infrastructure :: Lint and Formatting, defect, P2)
Developer Infrastructure
Lint and Formatting
Tracking
(firefox110 fixed)
RESOLVED
FIXED
| Tracking | Status | |
|---|---|---|
| firefox110 | --- | fixed |
People
(Reporter: standard8, Assigned: Sylvestre)
References
Details
Attachments
(2 files)
Bug 1802288 and bug 1802290 added checking of .idl and .webidl files for the file-whitespace and file-perm linters.
Unfortunately they missed updating the lists in taskcluster/ci/source-test/mozlint.yml which means that if a patch only touches one or both of those file types, then the linters won't run on CI.
Admittedly that is likely to be a rare case, but we should still fix it.
|
Assignee | |
Comment 1•2 years ago
|
||
|
||
Updated•2 years ago
|
Assignee: nobody → sledru
Status: NEW → ASSIGNED
|
|
Assignee | |
Comment 2•2 years ago
|
||
Depends on D165849
Pushed by sledru@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/594c9e464599
lint: file-perm/file-whitespace sync the extensions r=linter-reviewers,marco
https://hg.mozilla.org/integration/autoland/rev/d5e0f1af4d6a
ride along: lint: fix some windows CR r=linter-reviewers,marco
|
||
Comment 4•2 years ago
|
||
| bugherder | ||
https://hg.mozilla.org/mozilla-central/rev/594c9e464599
https://hg.mozilla.org/mozilla-central/rev/d5e0f1af4d6a
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
status-firefox110:
--- → fixed
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•