Skip to content
This repository has been archived by the owner on Dec 5, 2024. It is now read-only.

Provide PHP 8.1 support #8

Merged
merged 3 commits into from
Sep 2, 2021

Conversation

weierophinney
Copy link
Member

Prepares the component to work under PHP 8.1. The only changes necessary were:

  • Addition of PHP 8.1 to the PHP constraint
  • Removal of the laminas-config development dependency, and updating a single test that used laminas-config to use the equivalent ArrayObject for testing.
  • Updating an argument passed to ctype_space() to always pass a string, and thus prevent a deprecation notice.

- Updates PHP constraint to all PHP 8.1 series
- Removes laminas-zendframework-bridge requirement
- Removes "replace" configuration, and switches to "conflict" configuration, for indicating zend-text compatibility

Signed-off-by: Matthew Weier O'Phinney <matthew@weierophinney.net>
- Removes dependency on laminas-config; usage of `ArrayObject` in testing is equivalent to using `Laminas\Config\Config` instance.
- Updates argument passed to `ctype_space()` to ensure it is always a string, removing a deprecation notice.

Signed-off-by: Matthew Weier O'Phinney <matthew@weierophinney.net>
Signed-off-by: Matthew Weier O'Phinney <matthew@weierophinney.net>
@weierophinney weierophinney added this to the 2.9.0 milestone Sep 2, 2021
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 self-assigned this Sep 2, 2021
@Ocramius Ocramius merged commit 8879e75 into laminas:2.9.x Sep 2, 2021
@weierophinney weierophinney deleted the feature/php-8.1-support branch September 2, 2021 17:15
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants