-
Notifications
You must be signed in to change notification settings - Fork 91
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.
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
Containerize Application Using Docker #83
Comments
Please check the latest commit of my branch (gaurangkudale@08b0fc3) |
Thanks @gaurangkudale! I will look into it. |
Any updates or any suggestions @yannicnoller ?
…On Tue, Jun 20, 2023 at 5:49 AM Yannic Noller ***@***.***> wrote:
Thanks @gaurangkudale <https://github.com/gaurangkudale>! I will look
into it.
—
Reply to this email directly, view it on GitHub
<#83 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/APPZHRWNGD76PSU2P2YL5HTXMDT7ZANCNFSM6AAAAAAZLYVEEE>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
Hi @gaurangkudale! Thanks for pushing in this! I had no chance yet running your script. But from looking at your files, it looks good. At the end it would be great to also have one of the examples running, e.g., as a tutorial example and explain it in the corresponding README. Let me know if you want to have a comment on a specific aspect. Thanks! |
@yannicnoller Sure! |
Hmm not sure whether I understand it correctly. A PR basically shows what you have on your own branch (the one that you try to merge with ours). You can remove commits from your branch, if there is something that you like to change. However, we will likely simply squash your commits into one anyway. |
This pull request aims to containerize the application using Docker, providing an isolated and reproducible environment for easier deployment and scalability. Docker allows us to package the application and its dependencies into a lightweight, portable container, ensuring consistent behavior across different environments. (For Gradle-Build branch)
Changes To-Do:
@yannicnoller Please review this Feature and provide any feedback or suggestions. Thank you!
The text was updated successfully, but these errors were encountered: