feat: remove dependency doctrine/collections#167
Conversation
📝 WalkthroughWalkthroughDependency configuration update removing doctrine/collections from project requirements and bumping phpunit version constraints in composer.json to newer patch and minor versions. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #167 +/- ##
=========================================
Coverage 84.66% 84.66%
Complexity 329 329
=========================================
Files 33 33
Lines 1337 1337
=========================================
Hits 1132 1132
Misses 205 205 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Fix all issues with AI agents
In `@composer.json`:
- Line 48: The phpunit constraint "phpunit/phpunit": "^11.5.48 || ^12.5.6"
conflicts with the declared PHP baseline (>=8.2) because PHPUnit 12 requires PHP
>=8.3; either change the composer.json php requirement to "php": ">=8.3" to
allow PHPUnit 12, or remove the "|| ^12.5.6" alternative so the line becomes
"phpunit/phpunit": "^11.5.48" to stay compatible with PHP >=8.2—update the
composer.json "php" key or the phpunit version string accordingly.
Summary by CodeRabbit
✏️ Tip: You can customize this high-level summary in your review settings.