Skip to content

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

Merged
merged 5 commits into from
Jun 10, 2019

Conversation

dinogun
Copy link
Collaborator

@dinogun dinogun commented May 24, 2019

for adoptopenjdk official docker images at DockerHub.

@karianna karianna added this to the May 2019 milestone May 25, 2019
Copy link
Member

@karianna karianna left a 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

@karianna karianna modified the milestones: May 2019, June 2019 Jun 6, 2019
@dinogun
Copy link
Collaborator Author

dinogun commented Jun 10, 2019

@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.

@karianna
Copy link
Member

@dinogun Looks good but Appveyor is failing?

@dinogun
Copy link
Collaborator Author

dinogun commented Jun 10, 2019

Have opened issue #173 for tracking the appveyor issue. However this PR has nothing to do with that.

Copy link
Member

@karianna karianna left a comment

Choose a reason for hiding this comment

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

LGTM

@karianna karianna merged commit c8ac830 into master Jun 10, 2019
@dinogun dinogun deleted the add_dockerhub_script branch May 27, 2020 13:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants