Skip to content

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

4 participants