Latex formatting fixed + tests#8
Conversation
| - name: Set up Node.js | ||
| uses: actions/setup-node@v4 | ||
| with: | ||
| node-version: '22' |
There was a problem hiding this comment.
| node-version: '22' | |
| node-version: '20' |
| # Copy the frontend output from the first stage | ||
| COPY --from=frontend-builder /app/out /app/public | ||
|
|
||
| # Copy the backend app code to /app/app | ||
| COPY back/app /app/app | ||
| COPY back/entrypoint.sh /app/ | ||
| COPY back/requirements.txt /app/ | ||
|
|
There was a problem hiding this comment.
| # Copy the frontend output from the first stage | |
| COPY --from=frontend-builder /app/out /app/public | |
| # Copy the backend app code to /app/app | |
| COPY back/app /app/app | |
| COPY back/entrypoint.sh /app/ | |
| COPY back/requirements.txt /app/ | |
| # Copy the frontend output from the first stage | |
| COPY --from=frontend-builder /app/out /app/public |
Matteovanypersele
left a comment
There was a problem hiding this comment.
La correction du rendu LaTeX est nécessaire et bien ciblée (normalisation des délimiteurs, espacement des blocs et retours à la ligne), avec des tests associés couvrant des cas réels. Déployé et validé avec succès.
Le refactor du build Docker en multi-stage est pertinent : un seul build, plus d'image frontend-builder séparée.
J'ai laissé en suggestions 2 points mineurs non bloquant : alignement de la CI sur Node 20 (cohérent avec le Dockerfile) et suppression des COPY redondants dans back/Dockerfile (déjà couverts par COPY back/ .).
Je signale que front/Dockerfile est devenu orphelin depuis le multi-stage et peut être supprimé.
862fd72 to
4d649bd
Compare
Partially fixed #4 .