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 compatibility #671

Closed
wants to merge 1 commit into from
Closed

PHP 8.4 compatibility #671

wants to merge 1 commit into from

Conversation

danepowell
Copy link

@danepowell danepowell commented Nov 27, 2024

Goal

Fix #670

Design

Changeset

Remove E_STRICT and add explicit nullable types. This requires changing the minimum PHP version. Supposedly E_STRICT hasn't been used since PHP 8 so it should be safe to remove: https://php.watch/versions/8.4/E_STRICT-deprecated

Testing

I can only confirm this fixes the errors, I haven't tested it otherwise.

@danepowell
Copy link
Author

@imjoehaines @tomlongridge could you please review and merge this and cut a new release to support PHP 8.4?

@mclack
Copy link

mclack commented Dec 5, 2024

Hi @danepowell

Thanks for taking the time to raise this PR.

We are currently investigating and discussing support for PHP 8.4 internally, which includes evaluating this PR.

We'll make sure to post any updates here as they become available.

@mclack mclack added the needs discussion Requires internal analysis/discussion label Dec 5, 2024
@wiseguy1394
Copy link

@mclack Is there any progress on this?
At the moment bugsnag-symfony is a upgrade-stopper for us...

@clr182
Copy link

clr182 commented Jan 6, 2025

Hi @wiseguy1394

Unfortunately no update just yet but we'll be sure to keep you posted when we have more to share.

@hannah-smartbear
Copy link

Hi @danepowell and @wiseguy1394,

We just wanted to let you know that we have now released support for PHP 8.4 in v3.29.2 of bugsnag-php.
As such, we are going to close out this PR.

@hannah-smartbear hannah-smartbear added released This feature/bug fix has been released and removed needs discussion Requires internal analysis/discussion labels Jan 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
released This feature/bug fix has been released
Projects
None yet
Development

Successfully merging this pull request may close these issues.

PHP 8.4 support
5 participants