Linting Setup Rollout Progress
Temporary Document
This document tracks the rollout of the new linting setup. It will be deleted once all MRs are merged and the team is up to speed.
Track status for repos with branch new-pla-linting-and-formatting-setup.
Workflow Per Repo
- Claim row - add your name or "in progress"
git checkout new-pla-linting-and-formatting-setup- Rebase first (surface conflicts early):
git fetch
git rebase origin/<work branch> - Update Lefthook install:
- Remove
lefthookfrompackage.jsondevDependencies - Remove
postinstallhook that runslefthook install - Update lockfile (npm/yarn/pnpm) to drop Lefthook
- Replace
npx lefthook installin docs with:brew install lefthooklefthook install
- Remove
- Cosmetic-only check:
- Review commits for format-only diffs
- Drop or edit those commits (interactive rebase)
- Install hooks on host:
brew install lefthook # if needed
lefthook install - CI: If missing, add lint/analyze jobs to
.gitlab-ci.yml - Update table checkboxes
- Push:
git push --force-with-lease
Status
| Repo | Work Branch | Pkg Mgr | MR | Status | Hooks | Cosmetic | Rebase | Pushed | Installed |
|---|---|---|---|---|---|---|---|---|---|
| castoro | master | yarn | !43 | merged | x | x | x | x | x |
| coniglio | dev | yarn (no package.json) | !236 | merged | x | x | x | x | x |
| criceto | main | npm | !147 | open | |||||
| delfino | - | - | - | pending | |||||
| docs | main | yarn (packageManager) | !350 | merged | x | x | x | x | x |
| farfalla | dev | yarn | !12631 | merged | x | x | x | x | x |
| farfalla-https-guard | master | none | !26 | merged | x | x | x | x | x |
| farfalla-integrations | master | yarn | !74 | review | x | x | x | x | x |
| fenice | dev | yarn (packageManager) | !70 | review | |||||
| formica | main | npm | - | pending | |||||
| medusa | dev | yarn | !572 | review | x | x | x | x | x |
| micelio | main | yarn | !37 | merged | x | x | x | x | x |
| vito | main | npm | !6 | merged | x | x | x | x | x |
| volpe | dev | yarn | !382 | review | x | x | x | x | x |
| zoo | main | yarn | !22 | merged | x | x | x | x | x |
Notes
- docs: removed package-lock.json (was duplicate; uses yarn via packageManager)
- coniglio: has yarn.lock but no package.json
- farfalla-integrations: removed
.styleci.yml - No local tests needed; CI covers it
For config examples, see Linting References.