Fix newly reported PHPStan errors - #159
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 (1)
📝 WalkthroughWalkthroughThe change adds a PHPDoc array-shape annotation for ChangesPlugin data annotation
Estimated code review effort: 1 (Trivial) | ~2 minutes Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches 💡 1🛠️ Fix failing CI checks 💡
📝 Generate docstrings
🧪 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 |
There was a problem hiding this comment.
Pull request overview
This PR addresses newly reported PHPStan type issues by adding an inline type assertion for the WordPress.org plugin info response, clarifying the expected structure of the plugin version data within the WP-CLI checksum command.
Changes:
- Added a PHPDoc
@vararray-shape annotation to assert$plugin_datacontains aversion: stringkey after runtime validation. - Helps PHPStan understand
$plugin_data['version']is a string when used as a fallback version.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
Summary by CodeRabbit