Skip to content

Improve PHP version requirement to ^7.1 || ~8.0.0 || ~8.1.0 - #301

Closed
florianeckerstorfer wants to merge 3 commits into
mainfrom
fix-php-version-range
Closed

florianeckerstorfer wants to merge 3 commits into
mainfrom
fix-php-version-range

Conversation

@florianeckerstorfer

Copy link
Copy Markdown
Member

This came up in the comments to a recently closed PR and I would appreciate your input on this.

@florianeckerstorfer

Copy link
Copy Markdown
Member Author

Added PHP 8.2.x to version range, since the unit tests are currently passing for it

@Ocramius Ocramius left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Note: you may want to run composer update --lock

Comment thread composer.json Outdated
],
"require": {
"php": ">=7.1",
"php": "^7.1 || ~8.0.0 || ~8.1.0 || ~8.2.0",

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Careful with 8.2.0: I suggest holding off until it reaches RC, around october 2022.

Until then, any BC break is still possible.

@Ocramius

Copy link
Copy Markdown

Remember composer.lock

@Ocramius

Copy link
Copy Markdown

@florianeckerstorfer is this closed on purpose?

@florianeckerstorfer

Copy link
Copy Markdown
Member Author

@Ocramius Yes, it took a long while, but I followed your suggestion and excluded 8.2 for now. Merged #313 instead

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.

2 participants