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

feat: publish a docker image on release #853

Merged

Conversation

theofidry
Copy link
Member

I unfortunately could not take the username humbug which would match the namespace for the packagist package, and docker hub does not accept dashes nor less than 4 character usernames.

So the final name picked is boxproject.

This PR is a rebased version of #694.

Closes #694.
Closes #693.

Co-authored-by: Daniel Reiche daniel.reiche@check24.de

@theofidry theofidry merged commit 22c6ebe into box-project:main Jan 30, 2023
@theofidry theofidry deleted the build_publish_docker_image_on_release branch January 30, 2023 19:37
@theofidry
Copy link
Member Author

Thank you @BigBadBassMan, sorry it took so long

@BigBadBassMan
Copy link

@theofidry Thank you for continuing this. I'm ashamed to say, that I never got around to rework my PR and incorporate at least a barebone documentation - sorry. I had not dreamt that you would merge this, but I'm very thankful!

@theofidry
Copy link
Member Author

There is no shame there, it was more than enough :) In the end I only had to do some tweaks due to heavy rework of the CI and some CS changes!

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.

provide official docker images for box
3 participants