-
Notifications
You must be signed in to change notification settings - Fork 28
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
Migrate builds to GHA #94
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
(Consistency nits)
Thanks!
For reference, example builds:
@wiredfool Please could you review and add the credentials as secrets? |
Co-authored-by: Hugo van Kemenade <hugovk@users.noreply.github.com>
As a test/demonstration, I have also pushed the built images to my Docker account and tested here: https://github.com/nulano/Pillow/actions/runs/346068049/workflow#L44 |
Let's merge and add credentials later. Thanks! |
Sorry, this disappeared into my backlog, I’ll try to dig out the creds. |
Fixes #68
For python-pillow/Pillow#5028
The script is almost an exact copy from Travis, except updated to GHA syntax; there is just one small change due to file permissions:
sudo chmod a+w .
added at the start of the build and thesudo chown
command moved later.Docker login details need to be stored as a Github Secret in
DOCKER_USERNAME
andDOCKER_PASSWORD
.