Skip to content

Fix extension key and recommend pie install#162

Merged
divarvel merged 1 commit into
eclipse-biscuit:mainfrom
ptondereau:docs/php-install-fixes
May 9, 2026
Merged

Fix extension key and recommend pie install#162
divarvel merged 1 commit into
eclipse-biscuit:mainfrom
ptondereau:docs/php-install-fixes

Conversation

@ptondereau
Copy link
Copy Markdown
Contributor

@ptondereau ptondereau commented May 8, 2026

  • Fix composer key in PHP doc: ext-biscuit-phpext-biscuit_php (matches the actual extension name)
  • Recommend pie install ptondereau/biscuit-php as the preferred install method
  • Add a Symfony subsection pointing to ptondereau/biscuit-sf-bundle

@ptondereau ptondereau changed the title docs(php): fix extension key and recommend pie install Fix extension key and recommend pie install May 8, 2026
Comment thread docs/src/usage/php.md
{
"require": {
"ext-biscuit-php": "*"
"ext-biscuit_php": "*"
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

the mixed kebab/snake case is surprising. is this required by pie?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Yes this because the old CLI pecl support only underscore as a separator, so pie decided to have the same behavior

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Comment thread docs/src/usage/php.md
@ptondereau ptondereau force-pushed the docs/php-install-fixes branch from 5e6cae8 to f01350d Compare May 9, 2026 14:16
@divarvel divarvel merged commit ef0ba36 into eclipse-biscuit:main May 9, 2026
2 checks passed
@ptondereau ptondereau deleted the docs/php-install-fixes branch May 9, 2026 17:14
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