Divergence entre npm run format et npm run lint

[bonjourmauko]

Depends on #64

Amélioration technique

Contexte

Lorsque j'utilise npm run format et npm run lint,
je m'attends à ce qu'il n'ait pas de diff.

npm run format
git add -p
npm run lint
git diff | pbcopy

(rien)

Problème rencontré par les utilisateurs

Ce n'est pas le cas :

npm run format
git add -p
npm run lint
git diff | pbcopy

diff --git a/src/views/bouquets/BouquetEditView.vue b/src/views/bouquets/BouquetEditView.vue
index 82e1447..663fe6e 100644
--- a/src/views/bouquets/BouquetEditView.vue
+++ b/src/views/bouquets/BouquetEditView.vue
@@ -517,8 +517,8 @@ onMounted(() => {
               ]"
             >
               <DsfrAccordion
-                :title="datasetProperties.title"
                 :id="datasetProperties.id"
+                :title="datasetProperties.title"
                 :expanded-id="datasetProperties.id"
                 @expand="datasetProperties.id = $event"
               >
@@ -650,8 +650,8 @@ onMounted(() => {
               ]"
             >
               <DsfrAccordion
-                :title="datasetProperties.title"
                 :id="datasetProperties.id"
+                :title="datasetProperties.title"
                 :expanded-id="datasetProperties.id"
                 @expand="datasetProperties.id = $event"
               >

Proposition de solution au problème

Scénarios possibles (pas excluants)

• Ne formatter que les fichiers changés (via @streino in feat(social): commente un bouquet #180 (comment)

• Désactiver les règles qui rentrent en conflit

• Désactiver l'option --fix dans package.json

• Implémenter Fix erreurs de linting #64

• Intégrer npm run lint au pre-commit hook (dépend de Fix erreurs de linting #64)

Définition de terminé

• Qu'il n'y ait pas de diff

[abulte]

Je ferais:

1. Désactiver l'option --fix dans package.json ASAP
2. Fix erreurs de linting #64 quand le temps le permet
3. Intégrer npm run lint au pre-commit hook