Skip to content

chore(composer): require PHP 8.1 & WinterCMS 1.2.8+ - #26

Open
damsfx wants to merge 1 commit into
mainfrom
fix-requirements
Open

chore(composer): require PHP 8.1 & WinterCMS 1.2.8+#26
damsfx wants to merge 1 commit into
mainfrom
fix-requirements

Conversation

@damsfx

@damsfx damsfx commented Aug 26, 2026

Copy link
Copy Markdown

This pull request updates the required PHP version and adds a new dependency to the composer.json file to align with PR #22, which removes default controller views.

Dependency and compatibility updates:

  • Increased the minimum required PHP version from 7.2 to 8.1 to ensure compatibility with newer PHP features and security updates.
  • Added winter/wn-backend-module (version ^1.2.8 or dev-develop) as a required dependency, which may be needed for backend functionality.

Summary by CodeRabbit

  • Chores
    • Updated the minimum supported PHP version to 8.1.
    • Added a requirement for the Winter backend module version 1.2.8 or newer.

@coderabbitai

coderabbitai Bot commented Aug 26, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 1348579f-cea0-4c02-8df2-7e96f70121a1

📥 Commits

Reviewing files that changed from the base of the PR and between 65e7f4c and b9918a8.

📒 Files selected for processing (1)
  • composer.json

Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review.


Walkthrough

The Composer manifest now requires PHP 8.1 or newer. It also adds winter/wn-backend-module with the allowed versions ^1.2.8 and dev-develop.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Merge Risk: ⚪ Minimal · up to b9918

This is a localized dependency and compatibility update with no actionable merge-blocking risk remaining after normal checks and review.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly summarizes both primary changes: requiring PHP 8.1 or newer and updating the WinterCMS backend dependency to version 1.2.8 or newer.
Full details: Docstring Coverage

Explanation

No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (1 skipped: 1 unsupported.)

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix-requirements

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@damsfx
damsfx requested a review from LukeTowers August 26, 2026 18:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant