Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(lint): prettier behavior #77

Merged
merged 1 commit into from
Aug 16, 2024

Conversation

andriacap
Copy link

@andriacap andriacap commented Aug 9, 2024

fix(wip) prettier linter

  • add --check argument
  • add "node_modules" to .prettierignore
  • use same prettier version of app in package.json
  • add html and scss prettier linter
  • ajout d'une github action pour sauvegarder en cache l'installation des packages npm (npm install est ré éxécuter uniquement si le package-lock.json à été modifié) --> améliore le temps d'éxécution des github action

Reviewed-by: andriacap

@andriacap andriacap marked this pull request as ready for review August 12, 2024 08:55
@andriacap
Copy link
Author

Pour le lint frontend , soit on applique dans cette PR prettier avec le nouveau script dans le packahe.json :
"format": "prettier --config .prettierrc --ignore-path .prettierignore --write '**/*.{ts,html,scss}' ",

Soit on valide cette PR juste pour les modifications faites sur les github action et ensuite on créée une autre PR où on applique le lint du frontend .
Merci pour vos retours !

@edelclaux
Copy link
Contributor

edelclaux commented Aug 13, 2024

Pour le lint frontend , soit on applique dans cette PR prettier avec le nouveau script dans le packahe.json : "format": "prettier --config .prettierrc --ignore-path .prettierignore --write '**/*.{ts,html,scss}' ",

Soit on valide cette PR juste pour les modifications faites sur les github action et ensuite on créée une autre PR où on applique le lint du frontend . Merci pour vos retours !

ça formate plein de fichiers dans le front qui ne l'étaient pas. Je suis ok pour séparer en deux PRs. on ignore les erreurs les erreurs de lint du frontend à partir de désormais :)

Jolie manoeuvre pour récupérer la version de prettier du package-lock.json !

Je squash tes commits pour la lisibilité, et ça part en merge

@edelclaux edelclaux self-requested a review August 13, 2024 14:56
@edelclaux edelclaux changed the title fix(wip): prettier behavioir linter fix(lint): prettier behavioir linter Aug 13, 2024
@edelclaux edelclaux changed the title fix(lint): prettier behavioir linter fix(lint): prettier behavior Aug 13, 2024
@edelclaux edelclaux force-pushed the fix/prettier-behavior branch from 1af8935 to 9523025 Compare August 13, 2024 15:22
@edelclaux edelclaux merged commit 0bc9052 into PnX-SI:develop Aug 16, 2024
1 of 2 checks passed
@edelclaux edelclaux deleted the fix/prettier-behavior branch August 23, 2024 10:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants