betagouv / eac-api

API de la plateforme EAC. Distribue des données sur les Acteurs Culturels et leurs actions.
https://api.education-artistique-culturelle.fr/
MIT License
2 stars 1 forks source link

Refactor seeder #3

Closed rap2hpoutre closed 6 years ago

rap2hpoutre commented 6 years ago

Voilà les outils de seed, le README a été mis à jour. Je vais faire un peu de refactorisation après le merge (réorganisation des dossiers, suppression des dépendances inutiles et des fichiers obsolètes, etc.)

Tu peux tester en local, pour moi ça marche, je te laisse jeter un oeil de ton côté.

capture d ecran 2018-06-28 a 12 18 27
rap2hpoutre commented 6 years ago

(il faudra squasher pour merger)

rap2hpoutre commented 6 years ago

Je m'interroge sur https://github.com/betagouv/eac-api/blob/master/scripts/csv-schools.js#L2 Peut-être qu'on pourrait avoir un package.json dans le dossier scripts et un README.md ?

Bonne remarque, j'ai d'abord voulu faire comme ça d'ailleurs. Puis, je me suis dit qu'on considère actuellement eac-api comme une seule application. Donc avec des dépendances, dont certaines sont potentiellement partagées. J'ai cependant mis ces nouvelles dépendances (que je vais nettoyer asap) en save-dev, car elle ne sont nécessaire que pour les scripts. D'ailleurs, les scripts eux-même sont dans le pacakge.json de la racine aussi. Je pense que c'est lisible et "plus simple", dans un premier temps. Par contre, je vais un peu faire de ménage et réorganiser les fichiers pour que ce soit un peu plus expressif. Je ferais un commit de ça après merge.

Maintenant la vraie question, c'est : as-tu eu l'occasion de tester ? Chez moi ça marche et ça ressemble à ce qui est en prod. Donc si c'est bon pour toi, on merge, on livre et je fais un peu de refacto/reorga du repo (et on passe à la suite).