Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix runtime check of PHP version #5466

Merged
merged 1 commit into from
Jan 26, 2021
Merged

Conversation

keradus
Copy link
Member

@keradus keradus commented Jan 26, 2021

closes #5465

@keradus keradus added this to the 2.18.2 milestone Jan 26, 2021
@keradus keradus merged commit a7fc912 into PHP-CS-Fixer:2.18 Jan 26, 2021
@keradus keradus deleted the 2.18_Fix_runtime branch January 26, 2021 00:19
@coveralls
Copy link

Coverage Status

Coverage remained the same at 91.929% when pulling 37a9611 on keradus:2.18_Fix_runtime into 4a142ed on FriendsOfPHP:2.18.

mvorisek added a commit to mvorisek/PHP-CS-Fixer that referenced this pull request Jan 27, 2021
Copy link

@hason hason left a comment

Choose a reason for hiding this comment

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

Bug for PHP 8.0.*

} elseif (!defined('PHP_VERSION_ID') || \PHP_VERSION_ID < 50600 || \PHP_VERSION_ID >= 80100) {
} elseif (
!defined('PHP_VERSION_ID')
|| \PHP_VERSION_ID === 80000
Copy link

Choose a reason for hiding this comment

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

This is a bug. Version 8.0.* is supported.

Copy link
Member Author

Choose a reason for hiding this comment

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

@hason , no.
8.0.0 is not supported due to description from few lines below.
8.0.1+ is supported

This was referenced Mar 11, 2021
This was referenced Mar 11, 2021
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.

PHP8 Nullsafe operator gets deleted
3 participants