Skip to content

[ESLint] Nettoyage

AyM requested to merge (removed):master into master

@tcit, est-ce que tu peux me relire (sans merger sinon ça passe en prod) notamment ce qui concerne la réécriture des fonctions. C’est juste du nettoyage selon la config ESLint de Gitlab que tu m’as donnée. Le but est juste que je me familiarise avec les bonnes pratiques.

J’ai commité par type d’erreur au cas où il faudrait annuler certains trucs. En local ça fonctionne avec ou sans variable l$ pré-définie donc en théorie ça devrait passer en prod mais j’ai juste testé avec Firefox 58 et j’appréhende un peu la réception d’ES6 par les navigateurs de nos visiteurs. D’après les stats Piwik il y a vraiment très peu de visites avec des navigateurs non supportés.

Si c’est bon j’appliquerai les mêmes règles d’écritures pendant la refonte de de nav.js.

Après il ne reste que 4 erreurs : 3 sur le code de piwik qui à mon avis ne doit pas être modifié davantage et 1 du fait que l$ est une variable importable.

Merge request reports