Skip to content

feat(docker_launch_app): add dockerfile for start the app#75

Merged
MayuriXx merged 4 commits into
mainfrom
feat/docker_launch_app
Apr 20, 2026
Merged

feat(docker_launch_app): add dockerfile for start the app#75
MayuriXx merged 4 commits into
mainfrom
feat/docker_launch_app

Conversation

@MayuriXx
Copy link
Copy Markdown
Collaborator

@MayuriXx MayuriXx commented Apr 17, 2026

#74

Capture d’écran 2026-04-17 à 11 49 52

Copilot AI review requested due to automatic review settings April 17, 2026 09:51
@MayuriXx MayuriXx linked an issue Apr 17, 2026 that may be closed by this pull request
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds containerization support to run the Spring Boot application via Docker/Compose alongside the existing PostgreSQL service.

Changes:

  • Add an app service to docker-compose.yml to build/run the Spring Boot app connected to the postgres service.
  • Introduce a multi-stage Dockerfile to build the JAR with Maven and run it on a JRE base image.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 4 comments.

File Description
docker-compose.yml Adds an app service (builds from Dockerfile) and wires DB/Liquibase env + port mapping.
Dockerfile Multi-stage build to produce and run the Spring Boot fat JAR.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread Dockerfile Outdated
Comment thread Dockerfile
Comment thread docker-compose.yml
Comment thread docker-compose.yml Outdated
@MayuriXx MayuriXx force-pushed the feat/docker_launch_app branch from 5960524 to 8c2484a Compare April 17, 2026 10:04
Copy link
Copy Markdown
Collaborator

@profotoce59 profotoce59 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

AGTM ça me paraissait bizarre de mettre deux fois les variables d'environnement sur les deux services (app et postgres) mais pas moyen de les partager apparemment donc c'est bon.

Copilot AI review requested due to automatic review settings April 20, 2026 13:30
@MayuriXx MayuriXx force-pushed the feat/docker_launch_app branch from 8c2484a to 850d28a Compare April 20, 2026 13:30
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated 4 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread Dockerfile Outdated
Comment thread README.md
Comment thread README.md
Comment thread docker-compose.yml
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings April 20, 2026 13:35
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated 4 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread README.md
Comment thread README.md
Comment thread README.md
Comment thread Dockerfile
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings April 20, 2026 13:40
@sonarqube-xpeho
Copy link
Copy Markdown

Quality Gate passed Quality Gate passed

Issues
0 New issues
0 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarQube

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread Dockerfile
Comment thread README.md
Copy link
Copy Markdown
Collaborator

@Theo-lbg Theo-lbg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@MayuriXx MayuriXx merged commit 9119522 into main Apr 20, 2026
13 checks passed
@MayuriXx MayuriXx deleted the feat/docker_launch_app branch April 20, 2026 14:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add a Dockerfile to launch the application

5 participants