Prettier and ESLint, one umbrella
Adding a formatter and a linter to the whole codebase through one umbrella branch, with the reason for each decision recorded.
The codebase gained a formatter and a linter in one umbrella, with the formatting landing as its own commit and the lint rules set up on the recommended presets. Most of the umbrella’s work was deciding what each rule should do and recording the reason for each decision.
Formatting in its own commit#
Prettier ran as a single commit, so a reviewer could read the formatting change apart from the lint work.
Linting followed on the recommended presets, with the mechanical fixes applied and the rest reviewed by hand.
Suppressions with a reason#
Two suppressions around destructuring came out, since the lint configuration already ignores unused rest siblings. Each suppression carries its reason in a comment.
For the gating history behind this work, see every change gets a gate.