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

[TASK] Backport library to support PHP 7.0 and 7.1 #103

Merged
merged 1 commit into from
Dec 6, 2022

Conversation

andreaskienast
Copy link
Collaborator

The PHP code of this library is backported to additionally support both PHP 7.0 and PHP 7.1.

In order to achieve this, const visibility, nullable types and void types were removed and replaced with PHPDoc comments where applicable.

@andreaskienast andreaskienast force-pushed the bugfix/compat-php-70 branch 2 times, most recently from 7c772eb to 03aa925 Compare December 6, 2022 10:06
Copy link
Member

@ohader ohader left a comment

Choose a reason for hiding this comment

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

Please consider adjusting UPGRADING.md (types and version), as well as adding a note to README.md (that it was a back-port of v2 and is considered a transition version between 1.x and 2.x)

src/Behavior/NodeException.php Show resolved Hide resolved
src/Behavior/NodeException.php Show resolved Hide resolved
src/Sanitizer.php Show resolved Hide resolved
@ohader
Copy link
Member

ohader commented Dec 6, 2022

The title of this PR and commits should not be PHP < 7.2, but like ... support PHP 7.0 (since the 1.5 branch does not support any PHP 5 version).

The PHP code of this library is backported to additionally support both
PHP 7.0 and PHP 7.1.

In order to achieve this, const visibility, nullable types and `void`
types were removed and replaced with PHPDoc comments where applicable.
@andreaskienast andreaskienast changed the title [TASK] Backport library to support PHP < 7.2 [TASK] Backport library to support PHP 7.0 and 7.1 Dec 6, 2022
@ohader ohader merged commit 307cbd6 into v1.5 Dec 6, 2022
@ohader ohader deleted the bugfix/compat-php-70 branch December 6, 2022 10: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.

2 participants