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

Nettoyer le fichier .gitignore et supprimer les fichiers .gitkeep #146

Closed
6 tasks done
jpm-cbna opened this issue May 11, 2023 · 2 comments
Closed
6 tasks done

Nettoyer le fichier .gitignore et supprimer les fichiers .gitkeep #146

jpm-cbna opened this issue May 11, 2023 · 2 comments
Assignees
Labels
bug Something isn't working enhancement New feature or request

Comments

@jpm-cbna
Copy link
Contributor

jpm-cbna commented May 11, 2023

  • Supprimer dans le fichier .gitignore les entrées inutiles et ajouter les nouvelles
  • Supprimer les fichiers .gitkeep des dossiers contenant actuellement du contenu
  • Ajouter un fichier .editorconfig de base
  • Compléter le fichier frontend/.prettierrc avec le paramètre "trailingComma" : "es5" qui autorise l'ajout d'une virgule finale dans les objets, tableaux... (voir doc)
  • Supprimer le fichier frontend/tsconfig.json (contenant des paramètres spécifique à un utilisateur) et l'ajouter au fichier .gitignore
  • Nettoyer le fichier frontend/package.json des attributs inutiles ou vides
@jpm-cbna jpm-cbna added bug Something isn't working enhancement New feature or request labels May 11, 2023
@jpm-cbna jpm-cbna self-assigned this May 11, 2023
@andriacap
Copy link
Contributor

andriacap commented May 11, 2023

J'ai testé les config sur mon environnement de dev , ça a l'air d'être bien pris en compte.
Pour prettier j'avais testé un package npm qui permettait de pouvoir utiliser plus de paramètres dans le fichier de config prettier dont le "sort" des import comme ça peut être le cas avec isort coté backend (voir explicaiton de la librairie ici : https://github.com/trivago/prettier-plugin-sort-imports) .
Un autre exemple (https://catalincodes.com/posts/organise-imports-with-prettier-and-friends) qui permet de voir l'utilité de cette librairie . (Cela pourrait peut être généralisé dans GeoNature ?)

J'ai utilisé ces paramètres dans le fichier de config prettierc, qui permet de séparer et d'ordonner les imports suivant (import des package coeur à angular , puis les import de packages installés avec npm et enfin les imports de composants, services etc liés à l'application) :

    "importOrder": ["^components/(.*)$", "^[./]" ],
    "importOrderSeparation": true, 
    "importOrderSortSpecifiers": true,
    "importOrderParserPlugins": ["typescript", "decorators-legacy"]

@jpm-cbna
Copy link
Contributor Author

@andriacap Si on arrive à l'intégrer aux Github Actions, nous pouvons mettre en place le tri des imports dans les fichiers TypeScript que tu proposes.

@camillemonchicourt camillemonchicourt moved this from In Progress to Done in Refonte module EXPORT May 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request
Projects
Development

No branches or pull requests

3 participants