🧑‍💻 Add trivago

[FelixLgr]

Co-authored-by: Lucas Stoebner [email protected]

Close #2361

J'ai changé le .prettierrc.yaml en prettierrc car sur l'IDE webstorm, prettier n'arrive pas à trouver la config si c'est en yaml

PS: j'ai mit la page 404 en exemple pour que vous puissiez voir ce a quoi ca ressemble après un passage de prettier

[wiinxt]

J'ai merge dans une autre branch pour faire des tests

Il y a t'il une raison d'ajouter "trailingComma": "all", ?
Je trouve que ça ajoute beaucoup de virgule à des endroits pas forcement utile, exemple :

[wiinxt]

Pour "printWidth": 120, je sais pas trop non plus, quand on regarde un diff 80 caractères c'est bien je trouve, la pour moi ça dépasse de mon écran

et ça ne rend pas forcement plus lisible une grande ligne

@johangirod tu en pense quoi ?

[FelixLgr]

J'ai merge dans une autre branch pour faire des tests

Il y a t'il une raison d'ajouter "trailingComma": "all", ?

Je trouve que ça ajoute beaucoup de virgule à des endroits pas forcement utile, exemple :

Perso j'utilise cette option pour me faciliter le travail lorsque je duplique une ligne pour la modifier, ça m'évite de revenir sur la ligne d'avant pour ajouter la virgule

[wiinxt]

Pour les import c'est trop bien je trouve, le fix auto fonctionne, malheureusement il y a un problème avec le plugin de trivago et il n'ont pas l'air d'être super réactif
Le problème que j'ai rencontré viens du fichier api/source/index.ts qui ne voulais pas se formater, il y avait une erreur de @babel/parser dont la version est lock par le plugin trivago :

["ERROR" - 3:21:53 PM] Error formatting document.
["ERROR" - 3:21:53 PM] This experimental syntax requires enabling the parser plugin: "importAssertions". (8:37)

du coup je test avec le fork de https://github.com/IanVS/prettier-plugin-sort-imports que j'ai trouvé ici trivago/prettier-plugin-sort-imports#168 (comment)
Il a l'air d'être beaucoup plus réactif

edit: ça ne fonctionne pas non plus, je cherche ^^

[FelixLgr]

Pour les import c'est trop bien je trouve, le fix auto fonctionne, malheureusement il y a un problème avec le plugin de trivago et il n'ont pas l'air d'être super réactif

Le problème que j'ai rencontré viens du fichier api/source/index.ts qui ne voulais pas se formater, il y avait une erreur de @babel/parser dont la version est lock par le plugin trivago :

["ERROR" - 3:21:53 PM] Error formatting document.

["ERROR" - 3:21:53 PM] This experimental syntax requires enabling the parser plugin: "importAssertions". (8:37)

du coup je test avec le fork de https://github.com/IanVS/prettier-plugin-sort-imports que j'ai trouvé ici trivago/prettier-plugin-sort-imports#168 (comment)

Il a l'air d'être beaucoup plus réactif

edit: ça ne fonctionne pas non plus, je cherche ^^

Je sais que j'ai eu plusieurs problèmes sur des trucs bizarres.
Il fallait ajouter un truc du genre dans la configuration de prettier

"importOrderParserPlugins": ["typescript", "classProperties", "decorators-legacy"]

[wiinxt]

je viens de trouver la ligne qui cause cette erreur c'est les assert { type: 'json' } qu'il aime pas
On est trop dans le futur pour lui ^^

import openapi from './openapi.json' assert { type: 'json' }

[wiinxt]

Je sais que j'ai eu plusieurs problèmes sur des trucs bizarres.
Il fallait ajouter un truc du genre dans la configuration de prettier

"importOrderParserPlugins": ["typescript", "classProperties", "decorators-legacy"]

Merci, j'ai ajouté "importOrderParserPlugins": ["typescript", "jsx", "importAssertions"] et ça a l'air ok maintenant !
Je retest sur tout les fichiers

[wiinxt]

Du coup ça fonctionne super bien, j'ai juste retiré la différence "^react", car je ne vois pas trop l'utilité surtout que tout les packages react ne commence pas forcement par react... (par exemple @sentry/react, publicodes-react ou même styled-components qui en soit est un package react pour moi).

Avec ça on a les paquets npm, suivis des alias (pour site principalement), puis les fichiers locaux.