Skip to content

chore: Allow docker container to also build with gradle #478

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
dricazenck merged 1 commit into from
Jan 26, 2026
Merged

chore: Allow docker container to also build with gradle #478

dricazenck merged 1 commit into from
Jan 26, 2026

Conversation

@gpaOliveira
Copy link
Contributor

@gpaOliveira gpaOliveira commented Jan 20, 2026 •
edited
Loading

Description

  • Allowing the Java docker container (see Dockerfile) to also build with gradlew helps us to avoid having to install gradlew+Java directly in our machines
  • Having docker-compose to check for a healthy status allow us to know how much time it took to have a running app (today it's around 30s)
  • Finally, the scripts to start/cleanup docker containers allow us to not have to check the README all the time :)

Change Type

  • Bug Fix
  • New Feature
  • Code Refactor
  • Documentation
  • Test
  • Other

Pull request checklist

Please check if your PR fulfills the following requirements:

dricazenck
dricazenck reviewed Jan 20, 2026
View reviewed changes
@dricazenck dricazenck changed the title Allow docker container to also build with gradlew chore: Allow docker container to also build with gradle Jan 20, 2026
@dricazenck dricazenck marked this pull request as ready for review January 20, 2026 18:23
@dricazenck dricazenck requested a review from a team as a code owner January 20, 2026 18:23
dricazenck
dricazenck reviewed Jan 20, 2026
View reviewed changes
@gpaOliveira gpaOliveira force-pushed the multi-stage-build-with-gradlew branch from 74659a4 to fe5b2d3 Compare January 21, 2026 10:06
dricazenck
dricazenck approved these changes Jan 25, 2026
View reviewed changes
Copy link
Collaborator

@dricazenck dricazenck left a comment

Choose a reason for hiding this comment

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

🚀

@dricazenck dricazenck merged commit 032d3d9 into Women-Coding-Community:main Jan 26, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@dricazenck dricazenck dricazenck approved these changes

@nverbic nverbic Awaiting requested review from nverbic nverbic is a code owner automatically assigned from Women-Coding-Community/backend-team

@wilsonadenuga wilsonadenuga Awaiting requested review from wilsonadenuga wilsonadenuga is a code owner automatically assigned from Women-Coding-Community/backend-team

@silkenodwell silkenodwell Awaiting requested review from silkenodwell silkenodwell is a code owner automatically assigned from Women-Coding-Community/backend-team

@nora-weisser nora-weisser Awaiting requested review from nora-weisser nora-weisser is a code owner automatically assigned from Women-Coding-Community/backend-team

@goelsonali goelsonali Awaiting requested review from goelsonali goelsonali is a code owner automatically assigned from Women-Coding-Community/backend-team

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@gpaOliveira @dricazenck