extractor icon indicating copy to clipboard operation
extractor copied to clipboard

Allow Symfony 7 support

Open bocharsky-bw opened this issue 2 years ago • 9 comments

bocharsky-bw avatar Dec 26 '23 21:12 bocharsky-bw

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

Any volunteers to help?

bocharsky-bw avatar Jan 30 '24 13:01 bocharsky-bw

@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

rvanlaak avatar Jan 30 '24 16:01 rvanlaak

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?

bocharsky-bw avatar Jan 30 '24 16:01 bocharsky-bw

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

rvanlaak avatar Jan 31 '24 16:01 rvanlaak

Can you also add php-parser v5 maybe ?

Guite avatar Feb 07 '24 12:02 Guite

@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

rvanlaak avatar Feb 07 '24 12:02 rvanlaak

@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)

byhaskell avatar Mar 01 '24 10:03 byhaskell

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.

bocharsky-bw avatar Mar 01 '24 12:03 bocharsky-bw

Got it, thanks. I will try

byhaskell avatar Mar 01 '24 15:03 byhaskell

Closed in favor of #175

bocharsky-bw avatar Jun 11 '24 21:06 bocharsky-bw