Skip to content

Widen sebastian/diff constraint to allow v8/v9 To allow pest 5 - #12

Merged
veeqtoh merged 1 commit into
promptphp:0.xfrom
elliot-putt-spectre:widen-sebastian-diff-constraint
Jul 29, 2026
Merged

Widen sebastian/diff constraint to allow v8/v9 To allow pest 5#12
veeqtoh merged 1 commit into
promptphp:0.xfrom
elliot-putt-spectre:widen-sebastian-diff-constraint

Conversation

@elliot-putt-spectre

Copy link
Copy Markdown
Contributor

Summary

  • sebastian/diff is pinned to ^7.0.0, which forces consumers onto PHPUnit <=12 (PHPUnit 13 / Pest 5 require sebastian/diff ^9.0), making it impossible to depend on this package alongside Pest 5.
  • Widens the constraint to ^7.0.0|^8.0.0|^9.0.0. The only usage of the package, Differ/UnifiedDiffOutputBuilder in PromptDiffCommand, has an unchanged API across these versions.

Test plan

  • composer install still resolves with the default lowest constraint (sebastian/diff v7) — verified locally
  • composer require sebastian/diff:^9.0 -W resolves once the package's own dev dependencies (pest/phpunit) are also bumped to v5/v13 — left as a follow-up since it's a dev-only concern, not required for consumers

Allows consumers on PHPUnit 13 / Pest 5 (which require sebastian/diff ^9.0)
to install this package. The Differ/UnifiedDiffOutputBuilder API used in
PromptDiffCommand is unchanged across sebastian/diff v7-v9.
@elliot-putt-spectre elliot-putt-spectre changed the title Widen sebastian/diff constraint to allow v8/v9 Widen sebastian/diff constraint to allow v8/v9 To allow pest 5 Jul 29, 2026
@veeqtoh

veeqtoh commented Jul 29, 2026

Copy link
Copy Markdown
Member

@elliot-putt-spectre many thanks for this contribution.

@veeqtoh
veeqtoh merged commit 200cef8 into promptphp:0.x Jul 29, 2026
5 checks passed
@elliot-putt-spectre

Copy link
Copy Markdown
Contributor Author

@elliot-putt-spectre many thanks for this contribution.

No Problem thanks for the quick merge!

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