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

Added dockerfile #571

Merged
merged 6 commits into from
Nov 11, 2023
Merged

Added dockerfile #571

merged 6 commits into from
Nov 11, 2023

Conversation

norberttech
Copy link
Member

Change Log

Added

  • Dockerfile

Fixed

Changed

Removed

Deprecated

Security


Description

Refs: #511

Additionally, I applied for an opensource organization through dockerhub, once (if) we get it I will prepare GitHub actions that will build the image and push it after merge/release to dockerhub.

Dockerfile Outdated Show resolved Hide resolved
Dockerfile Outdated Show resolved Hide resolved
@flavioheleno
Copy link
Contributor

@norberttech why not host the docker images right here on GH?

to pull it, instead of org/img:tag you use ghcr.io/gh-org/img:tag, which is quite similar IMHO.

here how to automate publishing the images to GHCR using GHA: https://github.com/flavioheleno/watchr/blob/main/.github/workflows/release.yml#L68-L91

@norberttech
Copy link
Member Author

Honestly, I didn't think about it 🤦‍♂️ Yeah, I think we can start from here and if docker finally grants us open-source team license we can also publish to hub

@norberttech
Copy link
Member Author

Ok, I think I managed to make it work. Instead of fetching phar from github the Dockerfile will just grab the latest phar.

I tested it on my fork, everything worked just as expected.

@norberttech norberttech merged commit 881fd39 into flow-php:1.x Nov 11, 2023
11 checks passed
@norberttech norberttech mentioned this pull request Nov 11, 2023
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.

3 participants