Skip to content

optimise docker build and remove duplicate requirements.txt

Renaud Bardet requested to merge feature/js-refacto into main

J'ai modifié plusieurs choses dans le Dockerfile

  • ajouté l'installation des packets npm pour la compatibilité avec ces changements -> applications/risefor-front!22 (merged)
  • enlevé l'installation de sass via npm, le package systeme libsass ajoute deja ce qu'il faut
  • reordonné les étapes du Dockerfile pour mettre les requirements pip et npm en dernier, ca accelere le rebuild en local car les layers precedents sont mis en cache
  • le code n'est plus inclus comme un volume dans docker-compose mais via une copie dans le Dockerfile, ca me parraissait bizare que le Dockerfile ne copie pas les fichiers, si c'est une erreur de ma part je revert
  • execute le build docker depuis la racine, ce qui permet de garder risefor-front comme source de verité pour les requirements pip et package.json (Docker ne peut pas remonter les arborescences et je pense que c'est pour ca que requirements.txt était en doublon, mais si je me trompe je revert)

je fait cette requete de fusion en draft en attendant que la fusion associée dans risefor-front soit décidée

Edited by Renaud Bardet

Merge request reports