Skip to content

Require PHP >= 8.1.0 - #348

Open
MauricioFauth wants to merge 1 commit into
php:masterfrom
MauricioFauth:php-version-check
Open

MauricioFauth wants to merge 1 commit into
php:masterfrom
MauricioFauth:php-version-check

Conversation

@MauricioFauth

Copy link
Copy Markdown
Member

Adds a check for the minimum required PHP version. As enumerations are used, at least PHP 8.1 is required. It's also the required version for PhD.

Adds a check for the minimum required PHP version.
As enumerations are used, at least PHP 8.1 is required.
It's also the required version for PhD.
@alfsb

alfsb commented Sep 30, 2026

Copy link
Copy Markdown
Member

This ideia is good, but way over enginered. But please leave these two PRs open, white I try to cook something more simple with PHP_VERSION and PHP_BINARY.

@MauricioFauth

Copy link
Copy Markdown
Member Author

This ideia is good, but way over enginered.

I agree.

But please leave these two PRs open, white I try to cook something more simple with PHP_VERSION and PHP_BINARY.

Okay.

@hakre

hakre commented Oct 11, 2026 •

Copy link
Copy Markdown
Contributor

I'm undecided whether this is nice to have or only a maintenance burden, however as some food for thought: I think it would be of benefit when a version check is implemented, that the existing --version flag output pattern and behaviour is preserved.

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