Closed Bug 1596977 Opened 5 years ago Closed 4 years ago

[wpt-sync] Sync PR 20286 - [tools] Remove unused lint exceptions

Categories

(Testing :: web-platform-tests, task, P4)

Product:
Testing
Component:
web-platform-tests
Type:
task

Tracking

(firefox72 fixed)

Status:
RESOLVED FIXED
Milestone:
mozilla72
Tracking Flags:
Tracking Status
firefox72 --- fixed

People

(Reporter: mozilla.org, Unassigned)

References

(
URL
)

Details

(Whiteboard: [wptsync downstream])

Sync web-platform-tests PR 20286 into mozilla-central (this bug is closed when the sync is complete).

PR: https://github.com/web-platform-tests/wpt/pull/20286
Details from upstream follow.

Mike Pennisi <mike@mikepennisi.com> wrote:

[tools] Remove unused lint exceptions

These exceptions do not describe any infractions, so their presence
gives an inaccurate picture of where rules are being ignored.

It was tempting to extend the linter to enforce this whenever the --all flag is used, but there are a handful of generic exceptions that we'd have to remove (e.g. CR AT EOL: *.jpg). The convenience of not having to maintain those probably outweighs the benefit of enforcing utilization of every exception.

Then again, there may be a case for hard-coding generic exceptions like that into the application itself...

Pushed by wptsync@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/bc9206418833
[wpt PR 20286] - [tools] Remove unused lint exceptions, a=testonly
Test result changes from PR not available.

https://hg.mozilla.org/mozilla-central/rev/bc9206418833

Status: NEW → RESOLVED
Closed: 4 years ago
status-firefox72: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla72
You need to log in before you can comment on or make changes to this bug.