Skip to content

Conversation

@svycka
Copy link
Contributor

@svycka svycka commented Dec 21, 2020

Q A
New Feature yes

Summary

To be prepared for the december release of PHP 8.0, this repository has some additional TODOs to be tested against the new major version.

In order to make this repository compatible, one has to follow these steps:

  • Modify composer.json to provide support for PHP 8.0 by adding the constraint ~8.0.0
  • Modify composer.json to drop support for PHP less than 7.3
  • Modify composer.json to implement phpunit 9.3 which supports PHP 7.3+
  • Modify .travis.yml to ignore platform requirements when installing composer dependencies (simply add --ignore-platform-reqs to COMPOSER_ARGS env variable)
  • Modify .travis.yml to add PHP 8.0 to the matrix (NOTE: Do not allow failures as PHP 8.0 has a feature freeze since 2020-08-04!)
  • Modify source code in case there are incompatibilities with PHP 8.0

closes #8

Signed-off-by: Vytautas Stankus <[email protected]>
@svycka svycka marked this pull request as ready for review December 21, 2020 13:22
"symfony/dependency-injection": "^3.4.36 || ^4.4.2 || ^5.0"
},
"require-dev": {
"dms/phpunit-arraysubset-asserts": "^0.2.1",
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Maybe not the best solution but I did not want to change tests.

Copy link
Member

Choose a reason for hiding this comment

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

Makes perfect sense to me 👍

@Ocramius Ocramius self-assigned this Dec 23, 2020
@Ocramius Ocramius added this to the 1.5.0 milestone Dec 23, 2020
Copy link
Member

@Ocramius Ocramius left a comment

Choose a reason for hiding this comment

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

LGTM 👍 🚢

@Ocramius Ocramius merged commit c0ca6c1 into laminas:1.5.x Dec 23, 2020
@svycka svycka deleted the feature/php8.0-support branch December 23, 2020 13:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

PHP 8.0 support

2 participants