Update squizlabs/php_codesniffer requirement from ^3.11 to ^4.0#48
Update squizlabs/php_codesniffer requirement from ^3.11 to ^4.0#48dependabot[bot] wants to merge 1 commit intomainfrom
Conversation
|
@dependabot recreate |
73923e6 to
dd67690
Compare
Updates the requirements on [squizlabs/php_codesniffer](https://github.com/PHPCSStandards/PHP_CodeSniffer) to permit the latest version. - [Release notes](https://github.com/PHPCSStandards/PHP_CodeSniffer/releases) - [Changelog](https://github.com/PHPCSStandards/PHP_CodeSniffer/blob/4.x/CHANGELOG-4.x.md) - [Commits](PHPCSStandards/PHP_CodeSniffer@3.5.0...4.0.1) --- updated-dependencies: - dependency-name: squizlabs/php_codesniffer dependency-version: 4.0.1 dependency-type: direct:development ... Signed-off-by: dependabot[bot] <support@github.com>
dd67690 to
fdabd13
Compare
|
/gemini review and help me to fix this |
There was a problem hiding this comment.
Code Review
This pull request updates the squizlabs/php_codesniffer dependency to version 4.0. The review feedback recommends switching to the community-maintained phpcsstandards/php_codesniffer fork, as the original package is abandoned, and adjusting the alphabetical sorting of the package list.
| "phpstan/phpstan": "^2.1", | ||
| "phpunit/phpunit": "^11.5", | ||
| "squizlabs/php_codesniffer": "^3.11" | ||
| "squizlabs/php_codesniffer": "^4.0" |
There was a problem hiding this comment.
The squizlabs/php_codesniffer package has been officially abandoned in favor of phpcsstandards/php_codesniffer. Since you are performing a major version upgrade to 4.0, this is the ideal time to switch to the new vendor name to ensure long-term maintainability and alignment with the community standard.
Note that because your configuration has "sort-packages": true, you should also move this entry to its correct alphabetical position (between php-http/guzzle7-adapter and phpstan/phpstan) to maintain consistency.
| "squizlabs/php_codesniffer": "^4.0" | |
| "phpcsstandards/php_codesniffer": "^4.0" |
|
Looks like squizlabs/php_codesniffer is no longer updatable, so this is no longer needed. |
Updates the requirements on squizlabs/php_codesniffer to permit the latest version.
Release notes
Sourced from squizlabs/php_codesniffer's releases.
Changelog
Sourced from squizlabs/php_codesniffer's changelog.
... (truncated)
Commits
0525c73Merge pull request #1318 from PHPCSStandards/feature/changelog-4.0.14c09fb4Changelog for the 4.0.1 release7ed8ea2Merge pull request #1316 from PHPCSStandards/feature/tokenizer-php-switch-cas...b80dc24Various sniffs: update tests to safeguard handling ofcase/defaultwith P...6bd0af6Squiz/NonExecutableCode: bug fix - false positive with PHP close tag after ca...6d61496Squiz/SwitchDeclaration: make implied semicolon via close tag auto-fixable21c0e80PSR2/SwitchDeclaration: make implied semicolon via close tag auto-fixable6327f77Tokenizer/PHP: PHP close tag should be regarded as scope opener for switch ca...9397930Squiz/SwitchDeclaration: bug fix - fixer conflict for single line code (#1315)2c13a90Merge pull request #1308 from Soh1121/issue-968/fix-wrong-indentation-with-ar...