lint - html and scss are not included in CI linting

[edelclaux]

lint - html and scss are not included in CI linting

[TheoLechemia]

Salut,
Je ne suis personnellement pas fan du résultat des linter sur le html.. Mais le projet grossissant c'est peut être une étape nécessaire.
Si on le rend obligatoire, il faut pouvoir linter sans vscode, car tout le monde ne l'utilise pas. Comme on utilise déjà prettier pour les fichiers .ts, à mon sens il faut utiliser ça.

[edelclaux]

Salut, Je ne suis personnellement pas fan du résultat des linter sur le html.. Mais le projet grossissant c'est peut être une étape nécessaire. Si on le rend obligatoire, il faut pouvoir linter sans vscode, car tout le monde ne l'utilise pas. Comme on utilise déjà prettier pour les fichiers .ts, à mon sens il faut utiliser ça.

Je suis tombé sur un fichier avec des balises non fermées, et quelques autres soucis au milieu, ça me donne envie de linter moi :)

Ca me va prettier, j'ai vraiment pas de préférence.

Les paramètres que j'ai retenus: prettierrc enrichis de

• "singleAttributePerLine": true

Rappel des règles actuelles prettierrc:

"printWidth": 100,
"singleQuote": true,
"useTabs": false,
"tabWidth": 2,
"semi": true,
"bracketSpacing": true,
"trailingComma": "es5",

Vous en voyez d'autres ? @Pierre-Narcisi

[edelclaux]

J'ai fait une PR pour pouvoir jouer un peu et affiner les paramètres, ou changer la lib de linting si jamais

[camillemonchicourt]

Fait dans la 2.14.1