Ajout de prettier+husky pour reformater sur commit #49
Conversation
| @@ -7,7 +7,7 @@ parserOptions: | |||
| rules: | |||
| indent: | |||
| - error | |||
| - 4 | |||
| - 2 | |||
There was a problem hiding this comment.
...
tiens, un changement ^^
| "start": "serve --cors -p 3000", | ||
| "test": "exit 0", | ||
| "posttest": "npm run lint" | ||
| "precommit": "npm run beautify" |
There was a problem hiding this comment.
On a besoin d'eslint pour a minima pour les règles no-undef, eqeqeq, strict, no-unsafe-negation, no-unused-vars
There was a problem hiding this comment.
Oué, j'ai capturé ça en tant qu'élément de checklist dans l'issue.
There was a problem hiding this comment.
Pardon, j'avais raté la liste, dsl.
| @@ -41,6 +40,8 @@ | |||
| "devDependencies": { | |||
| "browserify": "^13.1.1", | |||
| "eslint": "^3.14.0", | |||
| "husky": "^0.13.1", | |||
There was a problem hiding this comment.
À plugger les git-hooks (n'importe lesquels) en tant que npm script.
|
C'est cool le reformattage mais ça flingue la possibilité de faire des commits partiels ( $ git status --short
MM client/js/finance/visitHierarchical.jsUne partie est staged, l'autre ne l'est pas. {
"scripts": {
"beautify": "prettier --write --single-quote -- --no-trailing-comma",
"precommit": "npm run beautify -- $(git diff --name-only --cached) | xargs git add"
}
}Le reformattage s'applique à tout :-( |
|
Spa encore ça. pre-commit ça sent pas bon et à part activer le reformatage au niveau de l'éditeur – ou en PR automatique – je vois pas encore de moyens de formater au bon moment. Je réouvrirai une PR avec avec |
Le but est de reformatter le code systématiquement de la même manière avant tout commit.
Le CI ne devrait pas avoir à foirer parce qu'un point-virgule est manquant.
eslint est toujours là pour prévenir d'erreurs en cours d'écriture (si un plugin eslint est activé dans l'éditeur de code).
Reste à faire :
prettier-eslintpour cabler eslint sur les autres règles de syntaxe (et non celles de formatage)fix #35