-
-
Notifications
You must be signed in to change notification settings - Fork 3
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 php8.3 version #26
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.
Thanks for looking into this, can you update the tests as well?
[ | ||
'8.3' | ||
].forEach(function (version) { |
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.
Can you change this into 8.4 and command the line out?
I've reviewd https://github.com/shivammathur/setup-php#tada-php-support indeed has 8.4 as an option |
That poses an interesting challenge. Personally I only use that option when the first alpha is out and I've started building it through https://github.com/wyrihaximusnet/docker-php and then update the upcoming release here. So going to |
I'd tend to agree with you, but as the README file states "The 4rth output is We can simply leave the upcoming list empty until first alpha is available or event maybe RC, but we can argue as well it is already in a separate configuration sot it is end user's responsibility for configuring it If you want I can empty the upcoming releases array again, and you can implement a new nightly flag |
Yes please as I rather have two since they are slightly different in expectations. Night would be wildwest no guarantee it will work. Upcoming is for more stability and preparing for the new release. I'll also update the readme wording to clarify the distinction. Can I tag you in that? |
I've just made the change, hope the tests pass Of course you can tag me |
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.
Awesome, thanks 👍 .
Will tag you in that nightly, and got an extension detection upgrade to do as well before releasing the next version |
Empty upcoming list until php8.4 rc comes