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

PHP 8.1 support #82

Merged
merged 5 commits into from
Sep 20, 2021
Merged

PHP 8.1 support #82

merged 5 commits into from
Sep 20, 2021

Conversation

boesing
Copy link
Member

@boesing boesing commented Sep 20, 2021

Q A
BC Break no (only on @internal marked classes)
New Feature yes

Description

Initial support for PHP 8.1.

The following changes were additionally made:

  • Support for symfony/console v4 was dropped
  • Compatibility classes (which were marked as @internal) for symfony/console v4 were dropped
  • Native types for classes (which are marked as @internal) were added to match symfony interface types
  • Some more "strict" inline type-hints were removed to match symfony typehints (were initially added with psalm support but were more strict than symfonys types)

Signed-off-by: Maximilian Bösing <2189546+boesing@users.noreply.github.com>
Signed-off-by: Maximilian Bösing <2189546+boesing@users.noreply.github.com>
This also removes some internal classes, adds native type-hints to internal classes which are meant to be BC breaks.

Signed-off-by: Maximilian Bösing <2189546+boesing@users.noreply.github.com>
@boesing boesing added this to the 1.2.0 milestone Sep 20, 2021
…n PHP 8.1

Signed-off-by: Maximilian Bösing <2189546+boesing@users.noreply.github.com>
… constraint

Signed-off-by: Maximilian Bösing <2189546+boesing@users.noreply.github.com>
composer.json Outdated Show resolved Hide resolved
src/Input/ParamAwareInput.php Show resolved Hide resolved
src/Input/StandardQuestionTrait.php Show resolved Hide resolved
@Ocramius Ocramius self-assigned this Sep 20, 2021
@Ocramius Ocramius merged commit 725d5db into laminas:1.2.x Sep 20, 2021
@Ocramius
Copy link
Member

Thanks @boesing!

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.

2 participants