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: enable PHP 8.1 #322

Closed
wants to merge 1 commit into from
Closed

feat: enable PHP 8.1 #322

wants to merge 1 commit into from

Conversation

dkarlovi
Copy link
Contributor

Create the 8.1 image set.

@jakzal
Copy link
Owner

jakzal commented Nov 30, 2021

Thanks for working on this.

We first need to get toolbox to support PHP 8.1: jakzal/toolbox#397

@dkarlovi
Copy link
Contributor Author

dkarlovi commented Dec 1, 2021

@jakzal I know, I wanted to set it up here so we have something to rebase. :)

@jakzal
Copy link
Owner

jakzal commented Dec 20, 2021

@dkarlovi just remembered you started it. Do you have time to rebase? :)

P.S. otherwise #334

@dkarlovi
Copy link
Contributor Author

Go ahead if you already have it done, no hrm done. 👌

@dkarlovi dkarlovi closed this Dec 20, 2021
@jakzal
Copy link
Owner

jakzal commented Dec 20, 2021

Thanks! I forgot about nightly builds by the way. Your PR helped me :)

@jakzal
Copy link
Owner

jakzal commented Dec 20, 2021

@dkarlovi I rebased your PR after all. Thought it's good to give credit when credit is due! ;)

@dkarlovi
Copy link
Contributor Author

That's nice of you, but don't make it create work for you, whichever way we get to PHP 8.1 images is awesome for me! 🍻

@jakzal jakzal mentioned this pull request Dec 20, 2021
@jakzal
Copy link
Owner

jakzal commented Dec 20, 2021

Ugh... looks like PHP 8.1.0 works while PHP 8.1.1 doesn't.

@dkarlovi
Copy link
Contributor Author

@jakzal it's always like that. 😆

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