Closed vinsag closed 2 months ago
Il y a déjà un fichier .dockerignore (conflit)
Il faudrait retirer les modifications sur les fichiers suivants :
A savoir avec les variables d'environnement sur NextJS : "Pour les app Next.js, les variables d'environnement sont traitées lors de la phase de build, remplaçant les références par leurs valeurs correspondantes. Seules les variables d'environnement commençant par NEXTPUBLIC sont exposées côté client, et les changements après le build nécessitent un nouveau build de l'application." Ce qui veut dire que si tu build ton image dans un github action, il faut avoir accès aux variables d'environnement.
Pour ce souci il y a cette note d'intéressant. si on veux rester runtime.
Note that dynamic lookups will not be inlined
Après passer (voir publier) les var NEXTPUBLIC* à un service de build ca ne me choque pas : ceux sont des valeurs qui seront de toute façon visible coté client. Si ce n'est pas le cas elle devrait pas etre avec ce prefix.
@antoineludeau : pour le yarn.lock c'est lorsque que j'ai fait l'update de ma branche avec master
@vinsag : ok mais tu ne devrais pas avoir de changement dans ce fichier si ? Je pense que tu peux retirer le README et le yarn.lock de cette PR.
Pour la migration mise à jour du dockerfile, env.sample et package.json