Closed Bug 1642825 Opened 1 year ago Closed 1 year ago

Prevent further use of black/whitelist in gecko with linting tests

Categories

(Firefox Build System :: Lint and Formatting, defect)

Product:
Firefox Build System
Component:
Lint and Formatting
Type:
defect

Tracking

(firefox79 fixed)

Status:
RESOLVED FIXED
Milestone:
mozilla79
Tracking Flags:
Tracking Status
firefox79 --- fixed

People

(Reporter: jkt, Assigned: Sylvestre)

References

(Blocks 1 open bug)

Details

Attachments

(2 files)

Bug 1642825 - mozlint: Add a linter to prevent further use of black/whitelist r=#linter-reviewers,gijs
47 bytes, text/x-phabricator-request
Details | Review
Bug 1642825 - mozlint: Run the rejected-words linter in the CI r=#linter-reviewers,gijs
47 bytes, text/x-phabricator-request
Details | Review
No description provided.
No longer blocks: remove-blackwhitelist
Component: General → Lint and Formatting
Product: Firefox → Firefox Build System

Note that I cannot do unit test because of it doesn't support regex payload
''Failed: could not resolve a lint function from''

Assignee: nobody → sledru
Status: NEW → ASSIGNED
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

https://hg.mozilla.org/mozilla-central/rev/831ed499fbb9
https://hg.mozilla.org/mozilla-central/rev/9ccfae770dc5

Status: ASSIGNED → RESOLVED
Closed: 1 year ago
status-firefox79: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla79
No longer blocks: 1647265
Depends on: 1647265
You need to log in before you can comment on or make changes to this bug.