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

PHP 8.4 #767

Open
wants to merge 6 commits into
base: main
Choose a base branch
from
Open

PHP 8.4 #767

wants to merge 6 commits into from

Conversation

dzuelke
Copy link
Contributor

@dzuelke dzuelke commented Nov 22, 2024

No blackfire or newrelic extension builds yet with support for 8.4. The phalcon extension is incompatible with 8.4 (they always get new releases some time later), and pcov needs a fix and release first: krakjoe/pcov#111. The imap extension has moved to PECL.

The heroku-20 stack is not getting this version because the stack is already deprecated.

GUS-W-17226361

We do not have support for PHP 5 or PHP 7.[012] anymore.
We do not have support for PHP 5 or PHP 7.0 anymore.
@dzuelke dzuelke requested a review from a team as a code owner November 22, 2024 17:06
@dzuelke dzuelke marked this pull request as draft November 22, 2024 17:17
There is no more E_STRICT for error_reporting, and session.sid_length is gone.

The imap extension has moved to PECL.

No blackfire or newrelic extension builds yet with support for 8.4. The phalcon extension is incompatible with 8.4 (they always get new releases some time later), and pcov needs a fix and release first: krakjoe/pcov#111

The heroku-20 stack is not getting this version because the stack is already deprecated.

GUS-W-17226361
We're bumping the plugin-api-version for the no lock file present case to allow Composer 2.8 which works with PHP 8.4 (to allow clean boots, and builds without deprecation errors).

This is a very inconsequential change for users that have no lock file, as no lock file means no package installs, plugins, etc.
@dzuelke dzuelke marked this pull request as ready for review November 22, 2024 23:12
else
HEROKU_PHP_DEFAULT_RUNTIME_VERSION="^8.0.0 <8.4"
HEROKU_PHP_DEFAULT_RUNTIME_VERSION="^8.0.0 <8.5"
Copy link
Member

@edmorley edmorley Nov 23, 2024

Choose a reason for hiding this comment

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

If I'm reading this correctly, apps with an unspecified PHP version can now get PHP 8.4 for the first time.

However, this isn't mentioned in the changelog? (The PHP/8.4.1 entry isn't sufficient to infer that IMO - particularly since it's a .1 release, and not 8.4.0, so doesn't make it clear that this is the first new 8.4 release)

I think an additional mention in the changelog might be worthwhile?

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