Prettier pre-commit hook

[abulte]

This adds a pre-commit hook to avoid committing unlinted/unformatted code.

Should avoid situations like in #154 where npm run format formatted some old code unrelated to the PR.

Give it a try, npm run prepare should be enough to get started.

NB: I can cleanup formatting changes from this PR if needed.

[netlify]

✅ Deploy Preview for ecospheres ready!

Name	Link
🔨 Latest commit	b4d3266
🔍 Latest deploy log	https://app.netlify.com/sites/ecospheres/deploys/654e120a0ae2290008abc500
😎 Deploy Preview	https://deploy-preview-155--ecospheres.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify site configuration.

[streino]

Testé et approuvé 💯 J'ai par contre des erreurs de linter sur l'existant 😅
+1 pour inclure le cleanup de l'existant dans cette PR.

Reste à régler le problème d'incohérence constaté entre les runs par @abulte et @YeLnatSs

[abulte]

J'ai par contre des erreurs de linter sur l'existant

Via npm run lint j'imagine ? Oui y'en a un partout. En revanche ce tooling devrait permettre de les régler petit à petit sur les fichiers qu'on touche si j'ai bien compris. D'ailleurs je ne connais pas ce tooling et j'ai fait ça à vue de nez, n'hésitez pas s'il y a mieux à faire !

[streino]

Via npm run lint j'imagine ?

Oui c'est ça. Par exemple, sur un fake commit modifiant DatagouvfrAPI.js

$ git commit -m "test"
✔ Preparing lint-staged...
⚠ Running tasks for staged files...
  ❯ package.json — 1 file
    ❯ *.{js,jsx,cjs,mjs,vue,json} — 1 file
      ✖ eslint --cache --fix [FAILED]
    ✔ *.{js,vue} — 1 file
↓ Skipped because of errors from tasks.
✔ Reverting to original state because of errors...
✔ Cleaning up temporary files...

✖ eslint --cache --fix:

.../ecospheres-front/src/services/api/DatagouvfrAPI.js
   91:13  error  Identifier 'entity_id' is not in camel case  camelcase
   92:34  error  Identifier 'entity_id' is not in camel case  camelcase
  102:14  error  Identifier 'entity_id' is not in camel case  camelcase
  103:34  error  Identifier 'entity_id' is not in camel case  camelcase
  146:16  error  Identifier 'entity_id' is not in camel case  camelcase
  148:24  error  Identifier 'entity_id' is not in camel case  camelcase

✖ 6 problems (6 errors, 0 warnings)

husky - pre-commit hook exited with code 1 (error)

Par contre, dans la mesure du possible / si c'est pas trop galère (selon les cas), ça serait cool de faire des PR séparées pour les correctifs de linting, plutôt que les grouper dans les PR de features (@YeLnatSs @edelagnier @bonjourmauko).

[abulte]

@streino ca va être galère les PR séparées je pense – mais c'est vrai que ça va polluer le code des features.

Sinon on peut:

• Désactiver le lint auto sur le hook de cette PR mais garder le format
• Faire une PR de lint fix global
• Réactiver le hook

[streino]

Sinon on peut:

* Désactiver le lint auto sur le hook de cette PR mais garder le format

* Faire une PR de lint fix global

* Réactiver le hook

Ça me semble bien comme approche. Tu peux faire le step 1 maintenant et on merge. On planifiera le step 2 par la suite 🙏

[edelagnier]

J'ai créé une issue pour pouvoir planifier l'étape 2 : #157

[edelagnier]

Application des préconisations du linter/formater, rien à signaler de mon côté

[streino]

On se sent déjà plus propres !
🚀