Set PHP 8.2 for dev - #132
tijmenbruggeman wants to merge 2 commits into
Conversation
|
Navigate logical layers of code changes, visualize relationships, and explore their blast radius. No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Essentials Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (2)
Included review availability: 4 reviews are currently available. Your included PR review attempts over the past 7 days set your current allowance at 5 reviews per hour. 📝 WalkthroughWalkthroughThe pull request changes the CI workflow matrices and Composer platform constraint from PHP 7.4 to PHP 8.2. ChangesPHP 8.2 runtime alignment
Priority: ⬇️ Low Estimated code review effort: 1 (Trivial) | ~5 minutes Change: Other Merge Risk: ⚪ Minimal · up to The change aligns Composer and CI with PHP 8.2, and the supplied evidence shows no concrete production or correctness risk blocking merge. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
A rabbit reads each line, Comment |
Currently we are using PHP 7.4 as the development version. Note, this is different from the runtime version which is >=5.3.0.
Out of scope
Summary by CodeRabbit