Openlibrary: Lintons notre JavaScript avec eslint !

Créé le 17 oct. 2018 · 5Commentaires · Source: internetarchive/openlibrary

Stylelint nous protège de divers problèmes CSS. Faisons de même pour notre JS.
eslint est le plus mature et fournit une liste de règles que nous pouvons utiliser si nous en jugeons le besoin.

npm install eslint --save-dev

Dans package.json, ajoutez une nouvelle commande

"lint:js": "eslint openlibrary/plugins/openlibrary/js/"

Ajouter un .eslintrc.json

{
    "root": true,
    "env": {},
    "globals": {
    },
    "rules": {
        "no-extra-semi": "warn"
    }
}

Nous allons définir les erreurs pour avertir pour le moment et dès que la configuration est fusionnée, nous devons nous coordonner avec quelqu'un pour exécuter les correctifs et fusionner afin d'éviter tout conflit de fusion !

[ ] npm run lint:js m'affichera les avertissements relatifs à la règle "no-extra-semi".
[ ] "npm run lint:js " se produit lorsque j'exécute "npm run test"
[ ] Ne corrigez aucune erreur dans le cadre de ce changement !

Good First Issue Linting JavaScript 1 hacktoberfest

Source

jdlrobson

Tous les 5 commentaires

Salut. Pouvez-vous m'assigner pour résoudre ce problème?

0x15F9 le 18 oct. 2018

Question : pourquoi ne pas utiliser l'un des préréglages pour eslint ? Comme https://standardjs.com ou https://github.com/airbnb/javascript

tamtamchik le 20 oct. 2018

À long terme, j'aimerais bien, mais le javascript est dans un si mauvais état, appliquer un tel préréglage serait très déroutant (révision de code, conflits de rebase) et un travail manuel (la plupart des préréglages ne peuvent pas être corrigés automatiquement). Je pense que l'ajout manuel de règles nous aide à l'attaquer petit à petit de manière cohérente et éducative (nous apprenons les problèmes dans notre code petit à petit).

J'ai pensé que nous pourrions utiliser le linting pour résoudre les problèmes les plus courants et les plus urgents (les lints qui peuvent provoquer des erreurs de code), puis appliquer à notre guise un préréglage (de notre choix) plus tard.

Est-ce que ça a du sens?

jdlrobson le 20 oct. 2018

👍1

Bien sûr, il le fait. La forme du code peut être très mauvaise.

Lorsque ce PR sera fusionné, je peux résoudre certains problèmes de peluches, il suffit de les nommer.
Commencera par no-extra-semi par exemple ;)

tamtamchik le 20 oct. 2018

👍1

@tamtamchik, je peux créer un problème plus tard, mais tout ce qui est sous "Erreurs possibles" snd et autofixable est un jeu équitable.

Nous devrions nous coordonner avec @mekarpeles pour exécuter et valider les corrections automatiques et faire une erreur eslint plutôt que d'avertir.

Je veux qu'on se débarrasse des fruits bas !

jdlrobson le 21 oct. 2018

🎉1

Cette page vous a été utile?

0 / 5 - 0 notes