Skip to content

Conversation

jrfnl
Copy link
Member

@jrfnl jrfnl commented Dec 7, 2023

PHP 8.2 | BCFile::getMemberProperties(): sync with upstream / add support for true pseudotype

As pulled in upstream PR squizlabs/PHP_CodeSniffer#3662 / PHPCSStandards/PHP_CodeSniffer#49.

Includes unit tests.

Support for the true type was previously already added to the Variables::getMemberProperties() method in PR #368.

Refs:

PHP 8.2 | BCFile::getMethodProperties(): sync with upstream / add support for true pseudotype

As pulled in upstream PR squizlabs/PHP_CodeSniffer#3662 / PHPCSStandards/PHP_CodeSniffer#49.

Includes unit tests.

Support for the true type was previously already added to the FunctionDeclarations::getProperties() method in PR #368.

Refs:

PHP 8.2 | BCFile::getMethodParameters(): sync with upstream / add support for true pseudotype

As pulled in upstream PR squizlabs/PHP_CodeSniffer#3662 / PHPCSStandards/PHP_CodeSniffer#49.

Includes unit tests.

Support for the true type was previously already added to the FunctionDeclarations::getParameters() method in PR #368.

Refs:

@jrfnl jrfnl added this to the 1.0.9 milestone Dec 7, 2023
@jrfnl jrfnl force-pushed the backcompat/bcfile-various-sync-add-support-for-true-type branch 3 times, most recently from df1458a to 998a769 Compare December 8, 2023 13:50
@jrfnl jrfnl marked this pull request as ready for review December 8, 2023 13:51
@jrfnl
Copy link
Member Author

jrfnl commented Dec 8, 2023

jrfnl added 3 commits December 8, 2023 15:11
…port for true pseudotype

As pulled in upstream PR squizlabs/PHP_CodeSniffer 3662 / PHPCSStandards/PHP_CodeSniffer 49.

Includes unit tests.
Includes moving some tests to allow the tests to still work as expected.

Support for the `true` type was previously already added to the `Variables::getMemberProperties()` method in PR 368.

Refs:
* squizlabs/PHP_CodeSniffer 3662
* PHPCSStandards/PHP_CodeSniffer 49
* PHPCSStandards/PHPCSUtils 368
…port for true pseudotype

As pulled in upstream PR squizlabs/PHP_CodeSniffer 3662 / PHPCSStandards/PHP_CodeSniffer 49.

Includes unit tests.

Support for the `true` type was previously already added to the `FunctionDeclarations::getProperties()` method in PR 368.

Refs:
* squizlabs/PHP_CodeSniffer 3662
* PHPCSStandards/PHP_CodeSniffer 49
* PHPCSStandards/PHPCSUtils 368
…port for true pseudotype

As pulled in upstream PR squizlabs/PHP_CodeSniffer3662 / PHPCSStandards/PHP_CodeSniffer 49.

Includes unit tests.
Includes moving some tests to allow the tests to still work as expected.

Support for the `true` type was previously already added to the `FunctionDeclarations::getParameters()` method in PR 368.

Refs:
* squizlabs/PHP_CodeSniffer 3662
* PHPCSStandards/PHP_CodeSniffer 49
* PHPCSStandards/PHPCSUtils 368
@jrfnl jrfnl force-pushed the backcompat/bcfile-various-sync-add-support-for-true-type branch from 998a769 to d1ddb2c Compare December 8, 2023 14:23
@jrfnl jrfnl merged commit 183a825 into develop Dec 8, 2023
@jrfnl jrfnl deleted the backcompat/bcfile-various-sync-add-support-for-true-type branch December 8, 2023 14:28
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.

1 participant