Skip to content

fix: required php version is mentioned in composer.json#81

Merged
OzzyCzech merged 1 commit intoOzzyCzech:mainfrom
Rotzbua:patch-1
Mar 25, 2026
Merged

fix: required php version is mentioned in composer.json#81
OzzyCzech merged 1 commit intoOzzyCzech:mainfrom
Rotzbua:patch-1

Conversation

@Rotzbua
Copy link
Copy Markdown
Contributor

@Rotzbua Rotzbua commented Mar 25, 2026

Avoid out of sync.

Copilot AI review requested due to automatic review settings March 25, 2026 14:35
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the README’s PHP requirement wording to avoid the documented PHP version drifting from the source-of-truth in composer.json.

Changes:

  • Replaces the explicit “PHP 8.1+” requirement with a reference to composer.json for the PHP version constraint.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

## Requirements

- PHP 8.1+
- PHP version see `composer.json`
Copy link

Copilot AI Mar 25, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The requirements bullet reads a bit ungrammatically. Consider rephrasing to something like “PHP version: see composer.json (the require.php constraint)” so it’s clear where to look and reads well in Markdown.

Suggested change
- PHP version see `composer.json`
- PHP version: see `composer.json` (the `require.php` constraint)

Copilot uses AI. Check for mistakes.
@OzzyCzech OzzyCzech merged commit 351f5b5 into OzzyCzech:main Mar 25, 2026
7 of 8 checks passed
@Rotzbua Rotzbua deleted the patch-1 branch March 25, 2026 14:49
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.

3 participants