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

add php8.3 version #26

Merged
merged 4 commits into from
Dec 3, 2023
Merged

add php8.3 version #26

merged 4 commits into from
Dec 3, 2023

Conversation

juliangut
Copy link
Contributor

Empty upcoming list until php8.4 rc comes

@WyriHaximus WyriHaximus added this to the v1.12.0 milestone Dec 2, 2023
Copy link
Owner

@WyriHaximus WyriHaximus left a 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?

Comment on lines 36 to 37
[
'8.3'
].forEach(function (version) {
Copy link
Owner

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?

@WyriHaximus WyriHaximus added the enhancement New feature or request label Dec 2, 2023
@juliangut
Copy link
Contributor Author

I've reviewd https://github.com/shivammathur/setup-php#tada-php-support indeed has 8.4 as an option

@WyriHaximus
Copy link
Owner

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 8.4 for the upcoming release isn't an option. However, adding it as a nightly toggle is the way to forward IMHO.

@juliangut
Copy link
Contributor Author

I'd tend to agree with you, but as the README file states "The 4rth output is
upcoming and will be populated with the upcoming but unreleased next minor or major version of PHP" as long as that version is available in shivammathur/setup-php and thus can be used it is the next unreleased version

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

@WyriHaximus
Copy link
Owner

I'd tend to agree with you, but as the README file states "The 4rth output is upcoming and will be populated with the upcoming but unreleased next minor or major version of PHP" as long as that version is available in shivammathur/setup-php and thus can be used it is the next unreleased version

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?

@juliangut
Copy link
Contributor Author

I've just made the change, hope the tests pass

Of course you can tag me

Copy link
Owner

@WyriHaximus WyriHaximus left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome, thanks 👍 .

@WyriHaximus WyriHaximus merged commit 5310ead into WyriHaximus:main Dec 3, 2023
11 checks passed
@WyriHaximus
Copy link
Owner

Will tag you in that nightly, and got an extension detection upgrade to do as well before releasing the next version

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.

2 participants