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

Composer: raise minimum PHPCS to 3.10.1 and sync some tests with upstream #629

Merged

Conversation

jrfnl
Copy link
Member

@jrfnl jrfnl commented Oct 7, 2024

Composer: raise the minimum supported PHPCS version to 3.10.1

... to benefit from fixes made to the File::findStartOfStatement() method and a tokenizer fix related to DNF types.

Includes updating references to the PHPCS version whenever relevant throughout the codebase.

Tests/BCFile::findStartOfStatement(): sync in new test from upstream [1]

See PR PHPCSStandards/PHP_CodeSniffer#502 / PHPCSStandards/PHP_CodeSniffer@467d284

Tests/BCFile::findStartOfStatement(): sync in new tests from upstream [2]

See PR PHPCSStandards/PHP_CodeSniffer#502 / PHPCSStandards/PHP_CodeSniffer@b82438f

Tests/BCFile::findStartOfStatement(): sync in new tests from upstream [3]

See PR PHPCSStandards/PHP_CodeSniffer#502 / PHPCSStandards/PHP_CodeSniffer@d3abcd6

Tests/BCFile::findStartOfStatement(): sync in test fixes from upstream [4]

See PR PHPCSStandards/PHP_CodeSniffer#509 / PHPCSStandards/PHP_CodeSniffer@83373f9

Tests/BCFile::find[Start|End]OfStatement(): sync in new tests from upstream [5]

See PR PHPCSStandards/PHP_CodeSniffer#509 / PHPCSStandards/PHP_CodeSniffer@a82f02e

... to benefit from fixes made to the `File::findStartOfStatement()` method and a tokenizer fix related to DNF types.

Includes updating references to the PHPCS version whenever relevant throughout the codebase.
@jrfnl jrfnl added this to the 1.1.0 milestone Oct 7, 2024
@jrfnl jrfnl merged commit b3d58ca into develop Oct 7, 2024
56 checks passed
@jrfnl jrfnl deleted the feature/tests-findstartendofstatement-sync-with-upstream branch October 7, 2024 04:24
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