Skip to content

Generators: show deprecation notice for deprecated methods #889

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

Merged
merged 1 commit into from
Apr 14, 2025

Conversation

jrfnl
Copy link
Member

@jrfnl jrfnl commented Mar 18, 2025

Description

These methods were soft deprecated in PHPCS 3.12.0 via PR #755 and will be removed in PHPCS 4.0.0.

This commit hard deprecates these in case extenders missed the changelog entry.

Includes perfunctory tests for the principle of the deprecation notices.

Suggested changelog entry

Added deprecation notices (hard deprecation) for:

  • Deprecated Generator method which will be removed in PHPCS 4.0.

@jrfnl jrfnl added this to the 3.x Last milestone Mar 18, 2025
@jrfnl jrfnl modified the milestones: 3.x Last, 3.13.0 Apr 13, 2025
These methods were _soft_ deprecated in PHPCS 3.12.0 via PR 755 and will be removed in PHPCS 4.0.0.

This commit hard deprecates these in case extenders missed the changelog entry.

Includes perfunctory tests for the principle of the deprecation notices.
@jrfnl jrfnl force-pushed the feature/generators-show-deprecation-for-methods branch from 9d4e2b6 to a72d2b5 Compare April 14, 2025 08:44
@jrfnl
Copy link
Member Author

jrfnl commented Apr 14, 2025

Rebased without changes before merging.

@jrfnl jrfnl merged commit 2ebcaed into master Apr 14, 2025
61 checks passed
@jrfnl jrfnl deleted the feature/generators-show-deprecation-for-methods branch April 14, 2025 09:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants