Open Bug 1433954 Opened 3 years ago Updated 3 years ago

[codespell] Use a blacklist instead of a whitelist + lint more directories

Categories

(Firefox Build System :: Lint and Formatting, enhancement, P2)

Product:
Firefox Build System
Component:
Lint and Formatting
Version:
3 Branch
Type:
enhancement

Tracking

(Not tracked)

People

(Reporter: ahal, Unassigned)

References

Details

In bug 1433912, I basically disabled the 'codespell' linter except on two directories. This was just because I needed to get the fix in quickly and didn't have time to do a thorough triage.

Now that the other bug is landed, we should take the time to run 'codespell' across the tree and enable it wherever it passes (or wherever there are only a handful of errors to fix).

We should also blacklist directories instead of whitelisting them. E.g:

include: ['.']
exclude:
    - servo
    - dom
    - layout
    - ...

This way new directories will automatically be linted.
Priority: -- → P2
Product: Testing → Firefox Build System
Version: Version 3 → 3 Branch
You need to log in before you can comment on or make changes to this bug.