[TECH] Formatage des fichiers de pix-orga avec Prettier.

[frinyvonnick]

🦄 Problème

Suite à cette PR, on va essayer Prettier dans pix-orga

🤖 Solution

Pour les fichiers hbs :
On utilise un plugin de ember-template-lint pour prettier : https://github.com/ember-template-lint/ember-template-lint-plugin-prettier

Pour les fichiers js :
On utilise Prettier via un plugin Eslint : https://github.com/prettier/eslint-plugin-prettier
Il recommande d'utiliser la configuration suivante qui désactive les règles conflictuelles : https://github.com/prettier/eslint-config-prettier.
Nous avons juste changer la règle singleQuote à true dans la configuration Prettier.

Pour tous les fichiers :
Nous avons aussi augmenté la valeur par défaut (80) de la largeur à ligne à 120.

🌈 Remarques

Prettier n'ajoute pas de nouvelle ligne à la fin des fichiers de template (hbs) car cela ajoute un text node vide dans le html généré (source: ember-cli/ember-cli#3440). Cependant nous avons gardé ce comportement dans les fichiers js (ce qui est le comportement par défaut de Prettier - non paramétrable).

Les retours de ember-template-lint ne sont pas visibles dans les fichiers hbs (testé sur VSCode et Neovim).

💯 Pour tester

Les instructions pour reproduire le problème, les profils de test, le parcours spécifique à utiliser, etc.

[pix-service]

I'm deploying this PR to these urls:

• App (.fr): https://app-pr3494.review.pix.fr
• App (.org): https://app-pr3494.review.pix.org
• Orga: https://orga-pr3494.review.pix.fr
• Certif: https://certif-pr3494.review.pix.fr
• Admin: https://admin-pr3494.review.pix.fr
• API: https://api-pr3494.review.pix.fr/api/

Please check it out!

[lisequesnel]

il y a des commits qui peuvent être fusionnés non ? par exemple c'est perturbant de voir tout qui change 1 fois dans le js car la taille des lignes est trop grandes, puis dans un commit ultérieur, la taille des lignes est augmentée, du coup tout re-change une 2e fois.

[jbuget]

Elle est atroce à review cette PR. Perso, je ne me vois pas revoir tous les fichiers. J'en ai déjà fait plus d'une quarantaine. Ma seul remarque avant de valider, c'esrt de bien remplacer les ="true" dans les template hbs par des ={{true}}.

[lisequesnel]

validé fonctionnellement ✅

[lisequesnel]

la gestion des "true" et "false" en chaîne a l'air de toujours fonctionner ✅

[frinyvonnick]

Pour info @octo-topi on a mis en place l'ADR 😉

[octo-topi]

Pour info @octo-topi on a mis en place l'ADR wink

👏
Ca va m'aider à débloquer cette PR pour Pix Admin #3358