Skip to content

Tokenizer: remove support for the deprecated @codingStandard annotation syntax#975

Merged
jrfnl merged 1 commit into4.xfrom
phpcs-4.0/feature/sq-1954-tokenizer-drop-old-style-annotations-support
Apr 14, 2025
Merged

Tokenizer: remove support for the deprecated @codingStandard annotation syntax#975
jrfnl merged 1 commit into4.xfrom
phpcs-4.0/feature/sq-1954-tokenizer-drop-old-style-annotations-support

Conversation

@jrfnl
Copy link
Copy Markdown
Member

@jrfnl jrfnl commented Apr 14, 2025 •
edited
Loading

Description

This annotation syntax was deprecated in PHPCS 3.2.0, which also introduced the new // phpcs:... syntax as a replacement.

Suggested changelog entry

Removed:

  • Support for the deprecated @codingStandard annotation syntax has been removed.
    • Use the phpcs: or @phpcs: syntax instead
      • Replace @codingStandardsIgnoreFile with phpcs:ignoreFile
      • Replace @codingStandardsIgnoreStart with phpcs:disable
      • Replace @codingStandardsIgnoreEnd with phpcs:enable
      • Replace @codingStandardsIgnoreLine with phpcs:ignore
      • Replace @codingStandardsChangeSetting with phpcs:set

Related issues/external references

Fixes squizlabs/PHP_CodeSniffer#1954

Related to #6

@gsherwood @jrfnl
Tokenizer: remove support for the deprecated @CodingStandard annotati…
847a11e 
…on syntax

This annotation syntax was deprecated in PHPCS 3.2.0, which also introduced the new `// phpcs:...` syntax as a replacement.

Fixes squizlabs/PHP_CodeSniffer 1954

Related to 6
@jrfnl jrfnl added this to the 4.0.0 milestone Apr 14, 2025
@jrfnl jrfnl merged commit 2f3d62e into 4.x Apr 14, 2025
53 of 54 checks passed
@jrfnl jrfnl deleted the phpcs-4.0/feature/sq-1954-tokenizer-drop-old-style-annotations-support branch April 14, 2025 14:28
@jrfnl jrfnl mentioned this pull request Apr 14, 2025
Closed
18 tasks
@jrfnl jrfnl mentioned this pull request Jul 15, 2025
Merged
This was referenced Sep 27, 2025
Merged
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@greg0ire greg0ire greg0ire approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

Core Component: File Core Component: Tokenizer Type: breaking change

Projects

None yet

Milestone

4.0.0

Development

Successfully merging this pull request may close these issues.

3 participants

@jrfnl @greg0ire @gsherwood