Skip to content

Allow Symfony 7 support #172

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

Closed
wants to merge 3 commits into from
Closed

Allow Symfony 7 support #172

wants to merge 3 commits into from

Conversation

bocharsky-bw
Copy link
Member

No description provided.

@bocharsky-bw
Copy link
Member Author

bocharsky-bw commented Jan 30, 2024

This is stuck, seems we need to move from Annotations to Attributes in case to support Symfony 7

Any volunteers to help?

@rvanlaak
Copy link
Member

@bocharsky-bw can you point in a direction, to be clear on the approach that the implementer could potentially follow? Does your comment also imply dropping doctrine/annotations and releasing a new minor or major of this library? https://github.com/php-translation/extractor/blob/master/composer.json#L16

@bocharsky-bw
Copy link
Member Author

Fairly speaking I didn't think about it yet, so I'm open to any good ideas :) But you're right, with more specific task it could be easier to pick this up, so here are my thoughts: I think it's time to drop PHP 7.2-8.0 support, that is ancient already so I think we can switch to PHP attributes completely and no need to support legacy annotations, and so we could drop that dependency completely. Though, I wonder if we should bump a major version for this, probably it's reasonable.

Thoughts?

@rvanlaak
Copy link
Member

The last major release was in 2019, so I think it's fine to do a major cleanup: https://github.com/php-translation/extractor/releases/tag/2.0.0

@Guite
Copy link
Contributor

Guite commented Feb 7, 2024

Can you also add php-parser v5 maybe ?

@rvanlaak
Copy link
Member

rvanlaak commented Feb 7, 2024

@Guite as PHP-Parser is not a hard requirement for Symfony 7, feel free to help by creating a separate PR to add support for PHP-Parser v5: https://github.com/nikic/PHP-Parser/blob/master/UPGRADE-5.0.md

@byhaskell
Copy link

@bocharsky-bw Hello. Tell me, please, where exactly are annotations used? I can’t find it, I’m trying to figure it out because without this bundle, entering text by hand is a lot of work)

@bocharsky-bw
Copy link
Member Author

bocharsky-bw commented Mar 1, 2024

This bundle provides some annotations https://github.com/php-translation/extractor/tree/master/src/Annotation , so those should be rewritten with Attributes because annotations support was dropped in Symfony which blocks upgrading this bundle as it requires annotations.

@byhaskell
Copy link

Got it, thanks. I will try

@bocharsky-bw
Copy link
Member Author

Closed in favor of #175

@bocharsky-bw bocharsky-bw deleted the sf7 branch June 11, 2024 21:44
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