Use ESLint instead of TSLint

[bfabio]

TSLint is deprecated, we should use ESLint instead.

[Ho3pLi]

ehi, è ancora da risolvere?

[bfabio]

Sì, vuoi provarci? 😉

[Ho3pLi]

si ci posso provare

[Ho3pLi]

@bfabio le regole che usava tslint non ci sono nel tsconfig.json perchè le prendeva dal tslint.json da
"extends": [ "italia-tslint-rules/strong" ]
giusto?

[bfabio]

@Ho3pLi esatto

Questo progetto è stato forkato da https://github.com/pagopa/io-spid-commons, nota che PagoPA ha fatto anche un altro repository per le regole di ESLint:

https://github.com/pagopa/eslint-rules

[Ho3pLi]

@bfabio ci ho lavorato un pochino e penso di aver finito, come controllo se è tutto ok?
Attualmente se lancio un lint-autofix non da errori e le regole sono quelle "nuove" (https://github.com/pagopa/eslint-rules).
Se provo a buildare però mi da errori di incompatibilità di versioni per le dipendenze.
Aggiorno il package.json con le nuove versione delle dipendenze?
Oppure avendo fatto la migrazione da tslint a eslint lascio tutto così?

[bfabio]

@Ho3pLi vuoi provare ad aprire una PR?

Il package.json (e yarn.lock) vanno aggiornati di sicuro, quantomeno per aggiungere eslint e togliere tslint.

[Ho3pLi]

Ho già tolto tslint e aggiunto eslint al package.json, adesso faccio la PR

[Ho3pLi]

@bfabio la versione di Node (10.14.1) usata nelle PR è incompatibile con uno dei moduli eslint. Ho provato a fixare cambiando versione del modulo, o anche togliendolo dalle dipendenze. Ma anche se i commit apportano modifiche, è come se le verifiche non fossero diverse. Cosa sto sbagliando? (scusa il disturbo davvero)

[bfabio]

@Ho3pLi innanzitutto grazie e scusa il ritardo nella risposta! 😞

Direi che possiamo prendere l'occasione per aggiornare la versione di node del progetto (e nella CI: https://github.com/italia/spid-express/blob/master/.github/workflows/ci.yml), che è troppo vecchia.

Ho aperto una nuova issue: #72

[Ho3pLi]

Finalmente! Allora mi ci rimetto al lavoro!