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: add missing trigger_deprecation calls to deprecated classes, methods #2265

Merged
merged 15 commits into from
Oct 17, 2024

Conversation

DominicLuidold
Copy link
Contributor

Q A
Bug fix? no
New feature? no
Deprecations? yes
Issues /

Adds missing deprecation triggers to classes and methods that have been deprecated with a @deprecated annotation.

@DominicLuidold DominicLuidold changed the title chore: Add missing trigger_deprecation calls to deprecated classes, methods fix: Add missing trigger_deprecation calls to deprecated classes, methods Apr 13, 2024
@DominicLuidold DominicLuidold changed the title fix: Add missing trigger_deprecation calls to deprecated classes, methods fix: add missing trigger_deprecation calls to deprecated classes, methods Jun 16, 2024
@DominicLuidold DominicLuidold force-pushed the trigger-deprecation-calls branch 2 times, most recently from dc3701e to 6f05acd Compare June 16, 2024 11:01
@DominicLuidold
Copy link
Contributor Author

@DjordyKoert Is there any way we could merge this PR? I unfortunately always have to keep up with the latest version changes (and update the PHPUnit baseline) after each release to keep the right version number (see e.g. b36a219). 😅

@DominicLuidold
Copy link
Contributor Author

I unfortunately always have to keep up with the latest version changes (and update the PHPUnit baseline) after each release to keep the right version number (see e.g. b36a219). 😅

If #2335 is merged before this, that's not necessary anymore (also, some changes would have to be reverted in this PR).

…s' into fork/DominicLuidold/trigger-deprecation-calls
@DjordyKoert
Copy link
Collaborator

Sorry this took way too long than it should have to merge.

Thank you for the time you spend on this @DominicLuidold ❤️

@DjordyKoert DjordyKoert merged commit 9587aa7 into nelmio:master Oct 17, 2024
15 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