Skip to content

chore(php8): drop deprecated INI settings in PHP 8.5 #0000 #18

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Ayesh
Copy link
Member

@Ayesh Ayesh commented Aug 24, 2025

We include a php.ini-production file, with sane defaults for all Docker images offered here.

In PHP 8.5, these two INI directives are deprecated:

  • report_memleaks: This directive has no effect on regular releases, certainly not on any of the official PHP Docker images, and is only effective on PHP debug builds.
  • disable_classes: The php.ini in this repo contains the same values as defaults (empty list of classes to disable), so we can safely drop references to them.

This PR suggests to remove them, to avoid triggering static analyzers unnecessarily.

Checklist

  • Breaking change: fix or feature that would cause existing functionality to not work as expected
  • I have read the Contributing doc
  • I have read the Creating and reviewing pull requests at LinkORB guide doc
  • Lint and unit tests pass locally with my changes
  • I have added/updated necessary documentation in the README.md or doc/ directories (if appropriate)

We include a php.ini-production file, with sane defaults for all Docker
images offered here.

In PHP 8.5, these two INI directives are deprecated:

 - [`report_memleaks`](https://php.watch/codex/report_memleaks#changes-php-8.5):
   This directive has no effect on regular releases, certainly not on any
   of the official PHP Docker images, and is only effective on PHP\
   debug builds.
 - [`disable_classes`](https://php.watch/codex/disable_classes#changes-php-8.5):
   The `php.ini` in this repo contains the same values as defaults
   (empty list of classes to disable), so we can safely drop references
   to them.

This PR suggests to remove them, to avoid triggering static analyzers
unnecessarily.
@Ayesh Ayesh force-pushed the fix/php5-ini-deprecations branch from 6ba1429 to 62ce753 Compare August 24, 2025 18:24
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.

1 participant