Skip to content

Deprecate annotation classes#187

Merged
bocharsky-bw merged 2 commits intophp-translation:masterfrom
mgiraud:deprecate-annotations
Mar 16, 2026
Merged

Deprecate annotation classes#187
bocharsky-bw merged 2 commits intophp-translation:masterfrom
mgiraud:deprecate-annotations

Conversation

@mgiraud
Copy link
Contributor

@mgiraud mgiraud commented Mar 16, 2026

No description provided.


namespace Translation\Extractor\Annotation;

trigger_deprecation('php-translation/extractor', '2.3', 'The "%s" class is deprecated with no replacement, it is now considered as a PHPDoc tag.', Desc::class);
Copy link
Member

Choose a reason for hiding this comment

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

How about this? I'm finding a way to push users to get rid of them faster :)

Suggested change
trigger_deprecation('php-translation/extractor', '2.3', 'The "%s" class is deprecated with no replacement, it is now considered as a PHPDoc tag.', Desc::class);
trigger_deprecation('php-translation/extractor', '2.3', 'The "%s" class is deprecated and will be removed soon. It is now considered as a PHPDoc tag.', Desc::class);

Copy link
Contributor Author

Choose a reason for hiding this comment

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

That's better, sounds like a call to action

Copy link
Member

@bocharsky-bw bocharsky-bw left a comment

Choose a reason for hiding this comment

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

Thank you for your help with further deprecating 👍🏼

@bocharsky-bw bocharsky-bw merged commit 8b219cf into php-translation:master Mar 16, 2026
5 checks passed
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.

2 participants