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

Remove unmaintained symfony libraries #1368

Closed
wants to merge 10 commits into from

Conversation

Shidarr
Copy link

@Shidarr Shidarr commented Oct 26, 2023

Q A
Type bug
BC Break no
Fixed issues

Summary

Drop unmaintained symfony 4.4 libraries

@stof
Copy link
Member

stof commented Oct 26, 2023

This should be submitted in the 3.7.x branch to remove support for Symfony 4.4 in the next minor version rather than in the next patch version IMO.

@Shidarr
Copy link
Author

Shidarr commented Oct 26, 2023

Good Idea. Maybe drop support for Versions 5.* also? Because if you update symfony to latest (6.3) Version, you will get some deprecation warnings.

@stof
Copy link
Member

stof commented Oct 26, 2023

I don't understand your proposal. Why wanting to drop support for 5.x (where 5.4 is still maintained for 1 year plus an extra year of security fixes) ? you're saying that you get some deprecation warnings when updating to 6.3, but I don't understand how this relates to the support of 5.x

@Shidarr Shidarr changed the base branch from 3.6.x to 3.7.x October 26, 2023 17:04
@Shidarr
Copy link
Author

Shidarr commented Oct 26, 2023

How to get rid of the deprecation warnings then?
They are mentioned here also: #1329
But the AsCommand Attribute is already added. Deleting the Parameter results in errors in older versions.
I can also add the Name in configure function and delete parameter. That should work too.

@derrabus
Copy link
Member

Did you check if this allows us to drop any compat code?

…so it works with old and new symfony/console Versions
…so it works with old and new symfony/console Versions
@Shidarr
Copy link
Author

Shidarr commented Oct 26, 2023

Ok I just updated the PR to use setName Function instead of $defaultName. This works for old and new symfony/console Versions. Maybe this is better? Then 5.4 is still supported.

@stof
Copy link
Member

stof commented Oct 26, 2023

the AsCommand attribute is supported starting in Symfony 5.3. So you don't need to drop support for 5.4.

@Shidarr
Copy link
Author

Shidarr commented Oct 26, 2023

Yes with my latest changes 5.4 does not need to be dropped and there is no deprecation warning anymore.

@derrabus
Copy link
Member

Dropping Symfony 4 was done in #1352 already. The changes you've made to the commands should not be necessary since the AsCommand attribute should suppress the deprecation regarding overriding the static properties.

@derrabus
Copy link
Member

Closing in favor of #1370.

@derrabus derrabus closed this Oct 29, 2023
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.

4 participants