Skip to content

issue/17 - Error Handler: Remove E_STRICT usage.#18

Merged
JJJ merged 1 commit intotrunkfrom
issue/17
Dec 10, 2025
Merged

issue/17 - Error Handler: Remove E_STRICT usage.#18
JJJ merged 1 commit intotrunkfrom
issue/17

Conversation

@JJJ
Copy link
Contributor

@JJJ JJJ commented Dec 10, 2025

Deprecated in PHP 8.4, this constant was not being used so it can safely be removed without causing any breakage.

Fixes #17.

Deprecated in PHP 8.4, this constant was not being used so it can safely be removed without causing any breakage.

Fixes #17.
Copilot AI review requested due to automatic review settings December 10, 2025 20:05
@JJJ JJJ changed the title Error Handler: Remove E_STRICT usage. issue/17 - Error Handler: Remove E_STRICT usage. Dec 10, 2025
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR removes the deprecated E_STRICT constant from the error handler in preparation for PHP 8.4 compatibility. The E_STRICT error level was deprecated in PHP 8.4 and this change ensures the codebase remains compatible with future PHP versions.

Key Changes

  • Removed the unused E_STRICT case block from the error handler switch statement

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@JJJ JJJ merged commit 2463f08 into trunk Dec 10, 2025
6 checks passed
@JJJ JJJ deleted the issue/17 branch December 10, 2025 20:09
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.

Deprecation notices in PHP 8.4 and 8.5

2 participants