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

PHP 8.2 | BCFile::get*() methods: sync with upstream / add support for true pseudotype #524

Merged

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
54 checks passed
@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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant