Open Bug 1433954 Opened 6 years ago Updated 3 months ago

[codespell] Use a blocklist instead of an allowlist + lint more directories

Categories

(Developer Infrastructure :: Lint and Formatting, enhancement, P2)

Product:
Developer Infrastructure
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
Product: Firefox Build System → Developer Infrastructure
Severity: normal → S3
Summary: [codespell] Use a blacklist instead of a whitelist + lint more directories → [codespell] Use a blocklist instead of an allowlist + lint more directories
You need to log in before you can comment on or make changes to this bug.