-
Notifications
You must be signed in to change notification settings - Fork 235
Add a script to generate the DockerHub config file #167
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
Conversation
…cker images at DockerHub.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Comments really - if they can't easily be done then LMK and I'll approve/merge this
Addressed earlier review comments.
@karianna Thanks for the comments, have updated the script to address your comments. Also added creation of the doc for the unofficial images as well. This should make it easy to copy-paste into the unofficial docker repo everytime we do releases / update the dockerfiles. |
@dinogun Looks good but Appveyor is failing? |
Have opened issue #173 for tracking the appveyor issue. However this PR has nothing to do with that. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
for adoptopenjdk official docker images at DockerHub.