Update phpunit/phpunit from 8.3.4 to 8.5.44 - #181
violinist-bot wants to merge 1 commit into
Conversation
WalkthroughBumped the PHPUnit development dependency in composer.json from version 8.3.4 to 8.5.44. No other files or public interfaces were modified. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Poem
Tip 🔌 Remote MCP (Model Context Protocol) integration is now available!Pro plan users can now connect to remote MCP servers from the Integrations page. Connect with popular remote MCPs such as Notion and Linear to add more context to your reviews and chats. ✨ Finishing Touches🧪 Generate unit tests
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. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. CodeRabbit Commands (Invoked using PR/Issue comments)Type Other keywords and placeholders
CodeRabbit Configuration File (
|
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (2)
composer.json (2)
12-16: Add an explicit PHP platform constraint.PHPUnit 8 requires PHP ≥7.2. Make the PHP version explicit in composer.json to fail fast on unsupported environments and keep CI/composer resolution clear.
Apply this diff:
"require": { + "php": "^7.2", "guzzlehttp/guzzle": "6.3.*", "symfony/dotenv": "4.3.*", "ext-json": "*" },If your CI supports newer PHP versions, adjust the constraint accordingly (e.g., "^7.4 || ^8.0").
23-23: Consider a looser constraint to receive future 8.5.x patches automatically.If you prefer automatic patch updates without editing composer.json each time, switch to a series constraint.
Proposed change:
- "phpunit/phpunit": "8.5.44", + "phpunit/phpunit": "8.5.*",Keep the exact pin if you value strict reproducibility over convenience (composer.lock already provides determinism in CI).
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
💡 Knowledge Base configuration:
- MCP integration is disabled by default for public repositories
- Jira integration is disabled by default for public repositories
- Linear integration is disabled by default for public repositories
You can enable these sources in your CodeRabbit configuration.
⛔ Files ignored due to path filters (1)
composer.lockis excluded by!**/*.lock
📒 Files selected for processing (1)
composer.json(1 hunks)
🔇 Additional comments (2)
composer.json (2)
17-25: Dev Dependencies Verified: No Prophecy Usage or Stray Transitive PackagesAll checks passed—no further action required:
- Tests scanned for any direct or trait-based Prophecy usage (
Prophecy,prophesize(),ProphecyTrait) and none was found.composer.lockwas inspected: no references to the removed transitive packages (phpdocumentor/*,phpspec/prophecy,webmozart/assert) remain.- Dev tool versions in
composer.lockmatch those incomposer.json(brianium/paratest 3.0.0; phpunit/phpunit 8.5.44).
23-23: Update PHPUnit Verification InstructionsThe pinned PHPUnit version is confirmed in composer.lock (8.5.44). To verify and run your tests locally, please install dependencies and check the version:
# Install dependencies (vendor/bin/phpunit will be created) composer install --no-interaction --prefer-dist # Verify the PHPUnit version ./vendor/bin/phpunit --version # Run the full test suite ./vendor/bin/phpunitEnsure that
./vendor/bin/phpunit --versionoutputs8.5.44and that your test suite passes both locally and in CI.
If you have a high test coverage index, and your tests for this pull request are passing, it should be both safe and recommended to merge this update.
Updated packages
Some times an update also needs new or updated dependencies to be installed. Even if this branch is for updating one dependency, it might contain other installs or updates. All of the updates in this branch can be found here:
Working with this branch
If you find you need to update the codebase to be able to merge this branch (for example update some tests or rebuild some assets), please note that violinist will force push to this branch to keep it up to date. This means you should not work on this branch directly, since you might lose your work. Read more about branches created by violinist.io here.
This is an automated pull request from Violinist: Continuously and automatically monitor and update your composer dependencies. Have ideas on how to improve this message? All violinist messages are open-source, and can be improved here.
Summary by CodeRabbit
Chores
Tests
No User-Facing Changes