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

Fixed formatting for new listing & adding github action support #51

Merged
merged 5 commits into from
Aug 22, 2021

Conversation

patbaumgartner
Copy link
Contributor

No description provided.

@patbaumgartner patbaumgartner changed the title Fixed formatting for new listing Fixed formatting for new listing & adding github action support Aug 22, 2021
@patbaumgartner
Copy link
Contributor Author

Adding github action support means automatically building docker images and pushing them to docker hub. Keep in mind you need to add DOCKER_USERNAME and DOCKER_PASSWORD to your github secretes.

Copy link
Owner

@brianleect brianleect left a comment

Choose a reason for hiding this comment

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

Looks good.

@brianleect brianleect merged commit f45ea14 into brianleect:master Aug 22, 2021
@brianleect
Copy link
Owner

Adding github action support means automatically building docker images and pushing them to docker hub. Keep in mind you need to add DOCKER_USERNAME and DOCKER_PASSWORD to your github secretes.

image

Seems to have some issue with access. Not sure if it's due to it not being linked to the current repo, but since both repos are public it shouldn't have been in a problem in theory i think.

docker/build-push-action#88
https://github.com/docker/build-push-action#usage

image

@patbaumgartner
Copy link
Contributor Author

Did you set DOCKER_USERNAME and DOCKER_PASSWORD or your docker hub account? If yes was it executed with the newest Dockerfile? Last solution idea: did you change the library name in the main.yml? maybe you do not have rights to access my library on docker hub :)

@brianleect
Copy link
Owner

brianleect commented Aug 22, 2021

image

Yep DOCKER_USERNAME and DOCKER_PASSWORD was already set up. Dockerfile used was the latest version in the repo if that's what you were referring to. I just changed library name in main.yml to brianleect/binance-pump-alerts.

image

I might be setting up docker wrongly. Thanks for the suggestions will see if I can figure out what I'm doing wrong.

Worst case I'll likely just leave it since I'm not actively using docker atm as well. The feature seems pretty neat though.

@patbaumgartner
Copy link
Contributor Author

patbaumgartner commented Aug 22, 2021 via email

@patbaumgartner
Copy link
Contributor Author

patbaumgartner commented Aug 22, 2021 via email

@brianleect
Copy link
Owner

brianleect commented Aug 22, 2021

you need to create a repo on Docker Hub https://hub.docker.com/repositories upfront. This is the one you push too. Best, Patrick On Sun, Aug 22, 2021 at 5:34 PM Patrick Baumgartner < @.***> wrote:

You are right, I did not create the repo previously. It works after I created the repo! Thanks for helping to debug!

image

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants