0xMiden / 0xMiden/web-sdk

vite-plugin isn't linted in CI

Aperta Adatta ai principianti
#265 2 commenti 0 reazioni 0 assegnatari Vedi su GitHub
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

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.