Skip to content

docs: preserve Bref php.ini scan directory#2144

Merged
mnapoli merged 1 commit into
brefphp:masterfrom
KairiYasumatsu:docs/preserve-php-ini-scan-dir
Jul 21, 2026
Merged

docs: preserve Bref php.ini scan directory#2144
mnapoli merged 1 commit into
brefphp:masterfrom
KairiYasumatsu:docs/preserve-php-ini-scan-dir

Conversation

@KairiYasumatsu

Copy link
Copy Markdown
Contributor

Summary

  • document how to add a custom PHP configuration directory without replacing Bref's default scan directory
  • add a serverless.yml example that keeps /opt/bref/etc/php/conf.d
  • warn that setting only the custom directory prevents bref.ini from loading

Why

PHP_INI_SCAN_DIR replaces PHP's configured scan directory. The current example suggests using only a custom path, which can stop Bref's /opt/bref/etc/php/conf.d/bref.ini from loading and cause bundled extension configuration to be lost.

The explicit default path follows the working configuration reported in #487.

Validation

  • npm run lint (0 errors; one existing no-explicit-any warning)
  • npm run build in website (93 pages generated, including /docs/environment/php)
  • verified the rendered HTML contains the configuration example and warning
  • git diff --check

Fixes #487

AI assistance

AI assistance was used to research and prepare this documentation change. I reviewed the final diff and validation results.

@mnapoli mnapoli left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

beautiful, thanks!

it's still in draft, I'll wait till you mark it ready to merge.

@KairiYasumatsu
KairiYasumatsu marked this pull request as ready for review July 21, 2026 02:06
@KairiYasumatsu

Copy link
Copy Markdown
Contributor Author

Thanks. I've marked it as ready for review.

@mnapoli
mnapoli merged commit 0139455 into brefphp:master Jul 21, 2026
8 checks passed
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.

Document that we should not use PHP_INI_SCAN_DIR

2 participants