Skip to content

T
third-party
Opened by  

Javascript lint style

Désolé pour la grosseur de la diff mais le style syntaxique du javascript est bien trop disparate dans toute la codebase et cela risque d'empêcher grandement les contributions futures si l'on vise à ce que le projet soit destiné à la plus de collaboration.

Commit par commit, la MR donne:

  • ajout d'un fichier package.json et de règles de style pour le code JS dans le fichier .eslintrc.js
  • ajout de l’exécution du linter automatiquement pour chaque commit via la CI (Continuous Integration) pour garder un style consistant pour le future
  • Commit le plus long et le moins intéressant car il ne contient que des modifications faites par une machine : c'est le résultat de npm run lint-fix qui demande à eslint to fixer automatiquement les règles de styles (e.g. espaces en trop, point-virgules manquants, ...)
  • fix errors not automatically fixable from eslint : ce sont toutes les erreurs de styles qui ne sont pas corrigeables automatiquement par la machine. Ce commit contient donc des modifications faites manuellement par moi et peut être intéressant à relire.
Edited by Paul
{{ resolvedDiscussionCount }}/{{ discussionCount }} {{ resolvedCountText }} resolved
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment