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

Provide support for PHP 8.1 #29

Merged

Conversation

weierophinney
Copy link
Contributor

This patch provides support for PHP 8.1.

In doing so, it also:

  • Removes support for PHP versions prior to 7.3.
  • Removes support for v2 releases of each of:
    • laminas-mvc
    • laminas-servicemanager
    • laminas-stdlib

To ensure compatibility, the patch:

  • Adds the convertDeprecationsToExceptions flag in the PHPUnit configuration
  • Adds #[ReturnTypeWillChange] attributes to IteratorAggregate::getIterator() definitions in the library
  • Updates to the laminas-coding-standard 2.3 version, but disables the rule that requires declare(strict_types=1) declarations (as they can introduce subtle BC breaks in minor versions).

- New PHP minimum version is 7.4.
- Adds PHP 8.1 to supported versions.
- Drops v2 support for each of:
  - laminas-mvc
  - laminas-servicemanager
  - laminas-stdlib
- Bumps minimum supported version of most libraries to versions known to work with PHP 8.1.

Signed-off-by: Matthew Weier O'Phinney <matthew@weierophinney.net>
Signed-off-by: Matthew Weier O'Phinney <matthew@weierophinney.net>
- Allows usage with PHP 8.1
- Exclude the rule requiring `declare(strict_types=1)` (adding strict types in a minor version can introduce subtle BC breaks)

Signed-off-by: Matthew Weier O'Phinney <matthew@weierophinney.net>
`IteratorAggregate::getIterator()` definitions

Signed-off-by: Matthew Weier O'Phinney <matthew@weierophinney.net>
@weierophinney weierophinney added this to the 1.5.0 milestone Jan 17, 2022
@weierophinney weierophinney added the Enhancement New feature or request label Jan 17, 2022
@weierophinney weierophinney linked an issue Jan 17, 2022 that may be closed by this pull request
@weierophinney weierophinney merged commit a06ec97 into laminas-api-tools:1.5.x Jan 17, 2022
@weierophinney weierophinney deleted the feature/27-php-8.1-support branch January 17, 2022 21:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Provide support for PHP 8.1.
1 participant