Skip to content
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

Switch docker push to be triggered on new release #311

Merged
merged 9 commits into from
Dec 6, 2023

Conversation

greenhat
Copy link
Member

@greenhat greenhat commented Sep 5, 2023

No description provided.

@reqlez
Copy link
Collaborator

reqlez commented Sep 5, 2023

After test is successful, don't forget there are 3 places with username/reponame in that action. Just heads up ;-)

@reqlez reqlez marked this pull request as ready for review September 7, 2023 06:02
@reqlez reqlez self-assigned this Sep 7, 2023
@reqlez
Copy link
Collaborator

reqlez commented Sep 7, 2023

Based on this run here, it's now working properly:
https://github.com/ergoplatform/oracle-core/actions/runs/6105865742

I changed it here to ergoplatform/oracle-core DockerHub repo now, that repo just needs to be created before we run this on release. @kushti heads up, and also set proper DOCKERHUB_USERNAME DOCKERHUB_TOKEN for ergoplatform account.

Otherwise, LGTM after the above 2 tasks are complete.

@reqlez reqlez requested a review from kushti September 7, 2023 06:15
@greenhat
Copy link
Member Author

greenhat commented Sep 8, 2023

@reqlez Great job! Thank you very much!

@reqlez
Copy link
Collaborator

reqlez commented Sep 8, 2023

@reqlez Great job! Thank you very much!

Happy to help! Now just waiting for @kushti to do his thing and we g2g!

@reqlez
Copy link
Collaborator

reqlez commented Oct 21, 2023

@kushti any luck setting this up?

@reqlez reqlez mentioned this pull request Dec 6, 2023
@kushti kushti merged commit 4ac8059 into develop Dec 6, 2023
7 checks passed
@kushti kushti deleted the docker-publish-test branch December 6, 2023 19:40
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.

3 participants