Closed
Bug 1444048
Opened 5 years ago
Closed 5 years ago
Codespell: ignore the ignored typos
Categories
(Developer Infrastructure :: Source Code Analysis, enhancement, P5)
Developer Infrastructure
Source Code Analysis
Tracking
(firefox61 fixed)
RESOLVED
FIXED
mozilla61
Tracking | Status | |
---|---|---|
firefox61 | --- | fixed |
People
(Reporter: Sylvestre, Assigned: Sylvestre)
References
Details
Attachments
(1 file)
codespell still shows some disabled warnings like: 68 warning iff ==> if | disabled due to valid mathematical concept (codespell) 147 warning gonna ==> going to | disabled because one might want to allow informal spelling (codespell) Moving to --quiet-level=4 silent them.
Comment hidden (mozreview-request) |
Comment 2•5 years ago
|
||
mozreview-review |
Comment on attachment 8957102 [details] Bug 1444048 - Codespell: ignore the ignored typos by moving to quiet-level=4 https://reviewboard.mozilla.org/r/226050/#review232808 Seems reasonable. r=Standard8.
Attachment #8957102 -
Flags: review?(standard8) → review+
Pushed by sledru@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/cc9ebc5403e2 Codespell: ignore the ignored typos by moving to quiet-level=4 r=standard8
Comment 4•5 years ago
|
||
Backed out changeset for linting opt failures on /builds/worker/checkouts/gecko/tools/lint/spell/__init__.py Treehder push with failures: https://treeherder.mozilla.org/#/jobs?repo=autoland&revision=cc9ebc5403e25a667d6588ac06b06ef0ecd51273&filter-resultStatus=testfailed&filter-resultStatus=busted&filter-resultStatus=exception&selectedJob=167437433 Failure log: https://treeherder.mozilla.org/logviewer.html#?job_id=167437433&repo=autoland&lineNumber=260 Backout link: https://hg.mozilla.org/integration/autoland/rev/34885bf65432ea0592e6652707f034691a20c3a8
Flags: needinfo?(sledru)
Comment hidden (mozreview-request) |
Pushed by sledru@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/244273b65767 Codespell: ignore the ignored typos by moving to quiet-level=4 r=standard8
Assignee | ||
Updated•5 years ago
|
Flags: needinfo?(sledru)
Comment 7•5 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/244273b65767
Status: NEW → RESOLVED
Closed: 5 years ago
status-firefox61:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla61
Updated•10 months ago
|
Product: Firefox Build System → Developer Infrastructure
You need to log in
before you can comment on or make changes to this bug.
Description
•