Closed
Bug 1642825
Opened 5 years ago
Closed 5 years ago
Prevent further use of black/whitelist in gecko with linting tests
Categories
(Developer Infrastructure :: Lint and Formatting, defect)
Developer Infrastructure
Lint and Formatting
Tracking
(firefox79 fixed)
RESOLVED
FIXED
mozilla79
| Tracking | Status | |
|---|---|---|
| firefox79 | --- | fixed |
People
(Reporter: jkt, Assigned: Sylvestre)
References
(Depends on 1 open bug, Blocks 1 open bug)
Details
Attachments
(2 files)
No description provided.
|
Reporter | |
Updated•5 years ago
|
No longer blocks: remove-blackwhitelist
|
|
Reporter | |
Updated•5 years ago
|
Blocks: remove-blackwhitelist
|
Assignee | |
Updated•5 years ago
|
Component: General → Lint and Formatting
Product: Firefox → Firefox Build System
|
|
Assignee | |
Comment 1•5 years ago
|
||
Note that I cannot do unit test because of it doesn't support regex payload
''Failed: could not resolve a lint function from''
|
||
Updated•5 years ago
|
Assignee: nobody → sledru
Status: NEW → ASSIGNED
|
|
Assignee | |
Comment 2•5 years ago
|
||
Depends on D78158
Pushed by sledru@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/831ed499fbb9
mozlint: Add a linter to prevent further use of black/whitelist r=Gijs,linter-reviewers,mossop,ahal
https://hg.mozilla.org/integration/autoland/rev/9ccfae770dc5
mozlint: Run the rejected-words linter in the CI r=Gijs,linter-reviewers,ahal
|
||
Comment 4•5 years ago
|
||
| bugherder | ||
https://hg.mozilla.org/mozilla-central/rev/831ed499fbb9
https://hg.mozilla.org/mozilla-central/rev/9ccfae770dc5
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
status-firefox79:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla79
|
||
Updated•5 years ago
|
|
|
Assignee | |
Comment 5•4 years ago
|
||
Example: It worked here: https://phabricator.services.mozilla.com/D102348?id=390131
|
||
Updated•3 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
•