chore: dependabot + PHP 8.4/8.5 CI matrix + PHPUnit 13 allowance - #5
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (3)
📝 WalkthroughWalkthroughThe pull request adds monthly Dependabot updates for Composer and GitHub Actions, expands the test matrix to PHP 8.4 and 8.5, and broadens the development PHPUnit version constraint. ChangesDependency maintenance
Estimated code review effort: 2 (Simple) | ~10 minutes 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
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 |
|
CI note: the initial run failed on all 5 matrix jobs at the live-smoke step, not the unit tests — the |
- Add .github/dependabot.yml (composer + github-actions, monthly) - Extend CI matrix with PHP 8.4 and 8.5 (SDK has zero runtime deps; full suite passes locally on PHP 8.5.8) - Allow phpunit ^13.0 in require-dev - verified locally: 16 tests / 80 assertions pass on PHPUnit 13.2.4; composer still resolves 10.x on PHP 8.1 runners composer validate --strict passes. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
28e86c9 to
d5fd521
Compare
What
^10.5 || ^11.0 || ^12.0 || ^13.0so newer PHP runners get the current majorVerification (local)
vendor/bin/phpunit→ OK (16 tests, 80 assertions)composer validate --strictpassesNo runtime code touched — the SDK has zero runtime dependencies (ext-curl/ext-json only).
🤖 Generated with Claude Code
Summary by CodeRabbit