Skip to content

Remove PECL build#21980

Open
iluuu1994 wants to merge 1 commit intophp:PHP-8.2from
iluuu1994:remove-pecl-build
Open

Remove PECL build#21980
iluuu1994 wants to merge 1 commit intophp:PHP-8.2from
iluuu1994:remove-pecl-build

Conversation

@iluuu1994
Copy link
Copy Markdown
Member

This build is not useful for multiple reasons:

  • Legitimate API changes keep making the build red. Frequently it takes 3rd party extension developers months to fix these issues.
  • Once we're annoyed enough by the red build we skip it and then don't revert it way after it has been fixed.
  • True accidental API changes are incredibly rare. I don't believe I've experienced one yet.
  • This build only runs for master, which is also the branch where accidental changes are least problematic.

@iluuu1994 iluuu1994 requested a review from TimWolla as a code owner May 8, 2026 07:44
@iluuu1994 iluuu1994 force-pushed the remove-pecl-build branch from 4b1317e to 969b7c1 Compare May 8, 2026 07:44
This build is not useful for multiple reasons:

- Legitimate API changes keep making the build red. Frequently it takes 3rd
  party extension developers months to fix these issues.
- Once we're annoyed enough by the red build we skip it and then don't revert it
  way after it has been fixed.
- True accidental API changes are incredibly rare. I don't believe I've
  experienced one yet.
- This build only runs for master, which is also the branch where accidental
  changes are least problematic.
@iluuu1994 iluuu1994 force-pushed the remove-pecl-build branch from 969b7c1 to 828f649 Compare May 8, 2026 07:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants