Closed
Bug 1830991
Opened 2 years ago
Closed 2 years ago
Make it so that violations of the test-manifest-alpha rule under browser/ are now errors instead of warnings
Categories
(Testing :: Mochitest, task)
Tracking
(firefox114 fixed)
RESOLVED
FIXED
114 Branch
| Tracking | Status | |
|---|---|---|
| firefox114 | --- | fixed |
People
(Reporter: mconley, Assigned: anwar.rhsl)
References
Details
Attachments
(1 file)
The other bugs blocking bug 1815451 have put all of the test manifests under browser/ into alphabetical order. Hooray! We now need to make sure they never go out of order again. We've got reviewbot warnings to help enforce this, but we can make this a little more strict now that browser/ is alphabetized.
By adding to tools/lint/test-manifest-alpha/error-level-manifests.yml, we can make violations of the rule under browser/ into errors.
So I think this means simply replacing this line with this:
- browser/**
|
Reporter | |
Updated•2 years ago
|
Assignee: nobody → anwar.rhsl
|
Assignee | |
Comment 1•2 years ago
|
||
Depends on D176929
Pushed by mconley@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/908855299d54
Make it so that violations of the test-manifest-alpha rule under browser/ are now errors instead of warnings. r=mconley
|
||
Comment 3•2 years ago
|
||
| bugherder | ||
Status: NEW → RESOLVED
Closed: 2 years ago
status-firefox114:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 114 Branch
You need to log in
before you can comment on or make changes to this bug.
Description
•