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

Add Support for PHP 8.4, Upgrade to Laminas Coding Standard 3.0 and Fix Compatibility with Symfony Console 7.1 #131

Merged
merged 1 commit into from
Nov 21, 2024

Conversation

fezfez
Copy link
Contributor

@fezfez fezfez commented Nov 17, 2024

No description provided.

@fezfez
Copy link
Contributor Author

fezfez commented Nov 17, 2024

the regression on latest job are due to this commit symfony/console@7cc16b1

after many try, all that i can do is tag symfony/console:7.1 in conflict

before $input->getStream() was evaluated each time (like now) but the value was stored once for all in QuestionHelper.

now $input->getStream() return null the second time is evaluated

test/ContainerResolverTest.php Outdated Show resolved Hide resolved
psalm-baseline.xml Outdated Show resolved Hide resolved
composer.json Outdated Show resolved Hide resolved
@samsonasik samsonasik added this to the 1.11.0 milestone Nov 18, 2024
@fezfez
Copy link
Contributor Author

fezfez commented Nov 18, 2024

@gsteel this should be good to go

@fezfez
Copy link
Contributor Author

fezfez commented Nov 21, 2024

@gsteel build is green even with symfony/console 7.1 :)

Copy link
Member

@gsteel gsteel left a comment

Choose a reason for hiding this comment

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

Nice 👌 Thanks @fezfez

@gsteel gsteel self-assigned this Nov 21, 2024
@gsteel gsteel changed the title Allow php 8.4 + composer bump --dev-only + laminas/laminas-coding-standard:^3 Add Support for PHP 8.4, Upgrade to Laminas Coding Standard 3.0 and Work Around Regression in Symfony Console 7.1 Nov 21, 2024
@gsteel gsteel merged commit 6309e11 into laminas:1.11.x Nov 21, 2024
16 checks passed
@gsteel gsteel mentioned this pull request Nov 21, 2024
@fezfez
Copy link
Contributor Author

fezfez commented Nov 21, 2024

@gsteel i dont think this is a regression in symfony 7.1 but a misuse in our side

@gsteel
Copy link
Member

gsteel commented Nov 21, 2024

Cool - I'll adjust the title

@gsteel gsteel changed the title Add Support for PHP 8.4, Upgrade to Laminas Coding Standard 3.0 and Work Around Regression in Symfony Console 7.1 Add Support for PHP 8.4, Upgrade to Laminas Coding Standard 3.0 and Fix Compatibility with Symfony Console 7.1 Nov 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants