Closed Bug 1455451 Opened 6 years ago Closed 6 years ago

Codespell: enable the autofix

Categories

(Developer Infrastructure :: Lint and Formatting, enhancement)

enhancement
Not set
normal

Tracking

(firefox61 fixed)

RESOLVED FIXED
mozilla61
Tracking Status
firefox61 --- fixed

People

(Reporter: Sylvestre, Assigned: Sylvestre)

References

Details

Attachments

(1 file)

Now that we have an ignore list, we can enable the autofix.

$ ./mach lint -l codespell -n dom/cache/CacheOpChild.cpp --fix
=> should not change anything (not typo and aParent won't be changed)

$ ./mach lint -l codespell -n dom/mathml/ --fix
=> will fix a typo in dom/mathml/nsMathMLElement.cpp
Comment on attachment 8969450 [details]
Bug 1455451 - Codespell: enable the autofix (with --fix)

https://reviewboard.mozilla.org/r/238214/#review244168

Awesome!
Attachment #8969450 - Flags: review?(ahalberstadt) → review+
Pushed by sledru@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/44607d1707a8
Codespell: enable the autofix (with --fix) r=ahal
https://hg.mozilla.org/mozilla-central/rev/44607d1707a8
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla61
Product: Firefox Build System → Developer Infrastructure
You need to log in before you can comment on or make changes to this bug.