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

Add Docker Github Action #2086

Merged
merged 2 commits into from Apr 25, 2021
Merged

Add Docker Github Action #2086

merged 2 commits into from Apr 25, 2021

Conversation

cooperlees
Copy link
Collaborator

@cooperlees cooperlees commented Apr 4, 2021

  • Build and upload arm64 + amd64 black images on push to master

This will need a DOCKERHUB_USERNAME and DOCKERHUB_TOKEN secrets set by someone with access.

Fixes #1914

- Build and upload arm64 + amd64 black images on push to master

This will need a `DOCKERHUB_USERNAME` and `DOCKERHUB_TOKEN` secrets set by someone with access.
@cooperlees cooperlees added T: enhancement New feature or request skip news Pull requests that don't need a changelog entry. labels Apr 4, 2021
@cooperlees
Copy link
Collaborator Author

Do we have a psf or python or something appropriate docker organization? Otherwise I'm happy to give other contributors access to my cooperlees/black

I'm torn on the News/Changelog entry - can add if people think we should.

@ewdurbin
Copy link
Member

ewdurbin commented Apr 6, 2021

@cooperlees we have python and pyfound orgs. psf is too short for a docker hub org name.

@ewdurbin
Copy link
Member

ewdurbin commented Apr 6, 2021

@cooperlees I added your user to the pyfound org on docker hub if you'd like to publish there.

on:
push:
branches:
- "master"
Copy link
Collaborator

Choose a reason for hiding this comment

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

Do we want to rename our master branch to main? It's quite easy nowadays (see https://github.com/github/renaming). This isn't blocking this PR but this reference reminded me that we should probably do this.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Yeah we should. Does it "rebase" / update nicely for people with existing checkouts etc? If so,. I can do now.

Copy link
Collaborator

Choose a reason for hiding this comment

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

No unfortunately:

Also, GitHub does not perform any redirects if users perform a git pull for the previous branch name

https://docs.github.com/en/github/administering-a-repository/renaming-a-branch#about-renaming-branches

Copy link
Collaborator

@JelleZijlstra JelleZijlstra left a comment

Choose a reason for hiding this comment

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

@cooperlees feel free to merge this when you feel it's ready

@cooperlees
Copy link
Collaborator Author

@cooperlees feel free to merge this when you feel it's ready

Need secrets added before I can merge. Do you have access?

@JelleZijlstra
Copy link
Collaborator

Nope, https://github.com/psf/black/settings 404s for me. It might be just Lukasz.

@cooperlees
Copy link
Collaborator Author

Nope, https://github.com/psf/black/settings 404s for me. It might be just Lukasz.

Bummer - Once secrets are added, we can merge this.

@cooperlees cooperlees merged commit e5490e9 into master Apr 25, 2021
@JelleZijlstra JelleZijlstra deleted the docker_upload branch April 25, 2021 18:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
skip news Pull requests that don't need a changelog entry. T: enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Official Docker Image
4 participants