Feat workflows#2
Merged
Merged
Conversation
…multi-stage builds
…ional development dependencies
devleesch001
added a commit
that referenced
this pull request
Sep 17, 2025
* Add CI workflows for frontend and backend, and update Dockerfile for multi-stage builds * Comment out test execution in CI workflows for frontend and backend * update Node.js version in CI workflow from 20 to 22 * Update CI workflow triggers to include workflow files for frontend and backend * Update CI workflows to specify context and Dockerfile paths for frontend and backend * Refine .dockerignore to include README.md and exclude unnecessary files * Remove production flag from npm ci command in Dockerfile * Update Dockerfile to copy from 'dist' directory and serve from 'dist' * Update Dockerfile to remove production flag from npm ci command and adjust base image * Update Dockerfile to clean npm cache after installation and add additional development dependencies * Implement graceful shutdown for the server and enhance shutdown logging * Update Dockerfile and package.json for improved build and configuration management * Refactor API URL configuration to use centralized config file * Fix indentation in App.test.tsx for improved readability --------- Co-authored-by: Alexis DEVLEESCHAUWER <a.devleeschauwer@pyres.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This pull request introduces significant improvements to both backend and frontend build, deployment, and configuration processes. The main changes include the addition of dedicated CI/CD workflows for each part of the stack, refactoring Dockerfiles for multi-stage builds, implementing a centralized frontend API configuration, and adding graceful shutdown logic for backend services.
CI/CD and Deployment Enhancements
.github/workflows/back.yml) and frontend (.github/workflows/front.yml) to automate build and Docker image publishing on relevant code changes.Dockerfile and Build Process Refactoring
back/Dockerfileandfront/Dockerfileto use Node.js 22 and multi-stage builds, resulting in smaller, production-ready images and cleaner separation of build and runtime environments..dockerignorefiles to backend and frontend to exclude unnecessary files from Docker build contexts, improving build performance.Frontend Configuration and API Integration
front/public/config/app-config.js, referencing it inindex.html, and refactoring API modules to use this config instead of hardcoded environment variables.Backend Service Reliability
back/src/index.tsto ensure clean closure of HTTP server, MongoDB, and Redis connections on termination signals.Dependency and Package Management
back/package.jsonandfront/package.jsonto ensure proper separation of dev and production dependencies, and updated lockfiles accordingly.