Skip to content

Conversation

@jrfnl
Copy link
Member

@jrfnl jrfnl commented Jul 20, 2020

Upstream PR squizlabs/PHP_CodeSniffer#3013 introduces a new $magicConstants tokens array to PHPCS, same as already existed in PHPCSUtils as Collections::$magicConstants since #106.

This PR backfills the PHPCS native array in the BCTokens class.

Includes unit test.

Upstream PR 3013 introduces a new `$magicConstants` tokens array to PHPCS, same as already existed in PHPCSUtils as `Collections::$magicConstants`.

This PR backfills the PHPCS native array in the `BCTokens` class.

Includes unit test.
@jrfnl jrfnl added this to the 1.0.0-alpha4 milestone Jul 20, 2020
@jrfnl jrfnl merged commit a647d9d into develop Jul 20, 2020
@jrfnl jrfnl deleted the bctokens/add-magicconstants branch July 20, 2020 23:25
jrfnl added a commit that referenced this pull request Dec 24, 2022
Use the PHPCS native `Tokens::$magicConstants` property or the `PHPCSUtils\BackCompat\BCTokens::magicConstants()` method instead.

Follow up on #106, #172, #311.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

2 participants