Skip to content

chore: remove ext-json dependency from all provider composer.json files#1433

Merged
atymic merged 1 commit intoSocialiteProviders:masterfrom
maks-oleksyuk:chore/remove-ext-json
Feb 21, 2026
Merged

chore: remove ext-json dependency from all provider composer.json files#1433
atymic merged 1 commit intoSocialiteProviders:masterfrom
maks-oleksyuk:chore/remove-ext-json

Conversation

@maks-oleksyuk
Copy link
Copy Markdown
Contributor

For composer.json file that had ext-json in the require section, this requirement is no longer necessary if already requires php ^8.0.

https://php.watch/versions/8.0/ext-json

Copy link
Copy Markdown

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 pull request removes the redundant ext-json dependency from all provider composer.json files across the repository. Since PHP 8.0, the JSON extension is always enabled and bundled with PHP core, making the explicit declaration unnecessary for packages requiring PHP ^8.0 or higher. This cleanup improves dependency accuracy and follows best practices for Composer dependency management.

Changes:

  • Removed "ext-json": "*" from 236 provider composer.json files
  • Removed "ext-json": "*" from the root composer.json file
  • All affected files require PHP ^8.0 or higher (versions 8.0, 8.1, 8.2, 8.3, or 8.4)

Reviewed changes

Copilot reviewed 266 out of 266 changed files in this pull request and generated no comments.

File Description
composer.json Removed redundant ext-json from root package dependencies
src/*/composer.json (236 files) Removed redundant ext-json from individual provider packages requiring PHP 8.0+

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

@atymic atymic merged commit 72c72d8 into SocialiteProviders:master Feb 21, 2026
8 checks passed
@atymic
Copy link
Copy Markdown
Member

atymic commented Feb 21, 2026

No need to tag new releases but thanks for the cleanup :)

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.

3 participants