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

Fix Symfony 6.1 deprecations #1472

Merged
merged 2 commits into from
Jun 8, 2022
Merged

Fix Symfony 6.1 deprecations #1472

merged 2 commits into from
Jun 8, 2022

Conversation

willemverspyck
Copy link
Contributor

Q A
Branch? 2.x
Bug fix? no
New feature? no
BC breaks? no
Deprecations? yes
License MIT
Doc

Since Symfony 6.1 setting the $defaultName is deprecated. Prefer to use "AsCommand" attribute, but that not supported yet in PHP ^7.1, so I moved it to the "setName" method.

Copy link
Member

@dbu dbu left a comment

Choose a reason for hiding this comment

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

things keep changing... thanks for the fix!

@dbu dbu merged commit 665b6a6 into liip:2.x Jun 8, 2022
@coveralls
Copy link

Coverage Status

Coverage decreased (-81.5%) to 0.0% when pulling 5c267bb on willemverspyck:2.x into f1466f4 on liip:2.x.

@dbu
Copy link
Member

dbu commented Jun 8, 2022

i merged the 2.x branch into the upcoming 3.x branch. that one has a more restrictive minimal php version requirement - if you have the time it would be great if you can do another PR for the 3.x branch to use the AsCommand attribute.

@willemverspyck
Copy link
Contributor Author

Thanks @dbu. I can add the attributes to version 3, but I see it still supports PHP 7.4 and attributes only work for PHP 8 and higher. Will support for PHP 7.4 be dropped for version 3?

@dbu
Copy link
Member

dbu commented Jun 9, 2022

ah dang. given that symfony 6 only supports php 8, i think the bundle should do the same. can you also do that change in the PR to the 3.x branch?

@willemverspyck
Copy link
Contributor Author

I did the changes to the 3.x branche and removed support of PHP 7.4. Also removed support of Symfony 4.4, because the attribute "AsCommand" is supported since Symfony 5.3. Not sure if you want these changes for 3.x.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants