vite-plugin isn't linted in CI
- Lingua principale
- TypeScript
- Stelle
- 1
- Fork
- 21
- Merge medio
- 12h 14m
- PR unite (30g)
- 41
Descrizione
Was digging through the lint setup and noticed packages/vite-plugin doesn't actually get linted anywhere in CI. Two things going on:
- the root eslint.config.js ignores packages/vite-plugin/**, so the repo-wide `eslint .` in the format job skips it
- nothing runs the package's own lint script either. make lint / rust-client-ts-lint only cover web_store, and there's no vite-plugin lint job like the react-sdk one
Looks like it was just missed when the per-package eslint configs were first set up. react-sdk and idxdb-store both got a local config plus wiring in the initial port, but vite-plugin only got the "lint": "eslint src" script in package.json, with no config for it to use and no CI hook. So the script's been dead since day one.
I've got a small fix ready that mirrors how react-sdk is handled: a local eslint.config.js for the package, plus a vite-plugin-lint target and CI job so it actually runs. The package source is already clean, so it passes with no code changes. Want me to open a PR for it?
Guida per i contributori
Apri la guida per i contributori
Direzione di ricerca
Esamina la directory packages/vite-plugin per vedere il suo package.json attuale e il file eslint.config.js mancante. Esamina come packages/react-sdk e packages/idxdb-store sono configurati con le loro proprie configurazioni di lint e job CI. La correzione prevede la creazione di una configurazione simile per vite-plugin e l'aggiunta di un target di lint nel Makefile o nella configurazione CI del progetto. Esegui il nuovo script di lint localmente per assicurarti che passi senza errori.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- eslint, typescript, vite
- Ambito
- ci-cd, tooling
- Tipo di issue
- Bug
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Stato di attività
- Attiva
- Chiarezza
- Specificata chiaramente
- Idoneità per principianti
- 70/100