Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
chore(deps-dev): Bump squizlabs/php_codesniffer from 3.9.1 to 3.9.2 (#5)
Bumps [squizlabs/php_codesniffer](https://github.com/PHPCSStandards/PHP_CodeSniffer) from 3.9.1 to 3.9.2. <details> <summary>Release notes</summary> <p><em>Sourced from <a href="https://github.com/PHPCSStandards/PHP_CodeSniffer/releases">squizlabs/php_codesniffer's releases</a>.</em></p> <blockquote> <h2>3.9.2 - 2024-04-24</h2> <h3>Changed</h3> <ul> <li>The Generic.ControlStructures.DisallowYodaConditions sniff no longer listens for the null coalesce operator. <a href="https://redirect.github.com/PHPCSStandards/PHP_CodeSniffer/issues/458">#458</a> <ul> <li>Thanks to <a href="https://github.com/rodrigoprimo">Rodrigo Primo</a> for the patch.</li> </ul> </li> <li>Various housekeeping, including improvements to the tests and documentation. <ul> <li>Thanks to <a href="https://github.com/fredden">Dan Wallis</a>, <a href="https://github.com/rodrigoprimo">Rodrigo Primo</a> and <a href="https://github.com/jrfnl">Juliette Reinders Folmer</a> for their contributions.</li> </ul> </li> </ul> <h3>Fixed</h3> <ul> <li>Fixed bug <a href="https://redirect.github.com/PHPCSStandards/PHP_CodeSniffer/issues/381">#381</a> : Squiz.Commenting.ClosingDeclarationComment could throw the wrong error when the close brace being examined is at the very end of a file. <ul> <li>Thanks to <a href="https://github.com/rodrigoprimo">Rodrigo Primo</a> for the patch.</li> </ul> </li> <li>Fixed bug <a href="https://redirect.github.com/PHPCSStandards/PHP_CodeSniffer/issues/385">#385</a> : Generic.CodeAnalysis.JumbledIncrementer improved handling of parse errors/live coding. <ul> <li>Thanks to <a href="https://github.com/rodrigoprimo">Rodrigo Primo</a> for the patch.</li> </ul> </li> <li>Fixed bug <a href="https://redirect.github.com/PHPCSStandards/PHP_CodeSniffer/issues/394">#394</a> : Generic.Functions.CallTimePassByReference was not flagging call-time pass-by-reference in anonymous class instantiations <ul> <li>Thanks to <a href="https://github.com/rodrigoprimo">Rodrigo Primo</a> for the patch.</li> </ul> </li> <li>Fixed bug <a href="https://redirect.github.com/PHPCSStandards/PHP_CodeSniffer/issues/420">#420</a> : PEAR.Functions.FunctionDeclaration could run into a blocking PHP notice while fixing code containing a parse error. <ul> <li>Thanks to <a href="https://github.com/jrfnl">Juliette Reinders Folmer</a> for the patch</li> </ul> </li> <li>Fixed bug <a href="https://redirect.github.com/PHPCSStandards/PHP_CodeSniffer/issues/421">#421</a> : File::getMethodProperties() small performance improvement & more defensive coding. <ul> <li>Thanks to <a href="https://github.com/jrfnl">Juliette Reinders Folmer</a> for the patch</li> </ul> </li> <li>Fixed bug <a href="https://redirect.github.com/PHPCSStandards/PHP_CodeSniffer/issues/423">#423</a> : PEAR.WhiteSpace.ScopeClosingBrace would have a fixer conflict with itself when a close tag was preceded by non-empty inline HTML. <ul> <li>Thanks to <a href="https://github.com/jrfnl">Juliette Reinders Folmer</a> for the patch</li> </ul> </li> <li>Fixed bug <a href="https://redirect.github.com/PHPCSStandards/PHP_CodeSniffer/issues/424">#424</a> : PSR2.Classes.ClassDeclaration using namespace relative interface names in the extends/implements part of a class declaration would lead to a fixer conflict. <ul> <li>Thanks to <a href="https://github.com/jrfnl">Juliette Reinders Folmer</a> for the patch</li> </ul> </li> <li>Fixed bug <a href="https://redirect.github.com/PHPCSStandards/PHP_CodeSniffer/issues/427">#427</a> : Squiz.Operators.OperatorSpacing would have a fixer conflict with itself when an operator was preceeded by a new line and the previous line ended in a comment. <ul> <li>Thanks to <a href="https://github.com/jrfnl">Juliette Reinders Folmer</a> for the patch</li> </ul> </li> <li>Fixed bug <a href="https://redirect.github.com/PHPCSStandards/PHP_CodeSniffer/issues/430">#430</a> : Squiz.ControlStructures.ForLoopDeclaration: fixed potential undefined array index notice <ul> <li>Thanks to <a href="https://github.com/jrfnl">Juliette Reinders Folmer</a> for the patch</li> </ul> </li> <li>Fixed bug <a href="https://redirect.github.com/PHPCSStandards/PHP_CodeSniffer/issues/431">#431</a> : PSR2.Classes.ClassDeclaration will no longer try to auto-fix multi-line interface implements statements if these are interlaced with comments on their own line. This prevents a potential fixer conflict. <ul> <li>Thanks to <a href="https://github.com/jrfnl">Juliette Reinders Folmer</a> for the patch</li> </ul> </li> <li>Fixed bug <a href="https://redirect.github.com/PHPCSStandards/PHP_CodeSniffer/issues/453">#453</a> : Arrow function tokenization was broken when the return type was a stand-alone <code>true</code> or <code>false</code>; or contained <code>true</code> or <code>false</code> as part of a union type. <ul> <li>Thanks to <a href="https://github.com/jrfnl">Juliette Reinders Folmer</a> for the patch</li> </ul> </li> </ul> <h3>Other</h3> <ul> <li><a href="https://eslint.org/blog/2024/04/eslint-v9.0.0-released/#flat-config-is-now-the-default-and-has-some-changes">ESLint 9.0</a> has been released and changes the supported configuration file format. The (deprecated) <code>Generic.Debug.ESLint</code> sniff only supports the "old" configuration file formats and when using the sniff to run ESLint, the <code>ESLINT_USE_FLAT_CONFIG=false</code> environment variable will need to be set when using ESLint >= 9.0. For more information, see <a href="https://redirect.github.com/PHPCSStandards/PHP_CodeSniffer/issues/436">#436</a>.</li> </ul> <hr /> <h3>Statistics</h3> <p><strong>Closed</strong>: 0 issues <strong>Merged</strong>: 22 pull requests</p> <p>If you like to stay informed about releases and more, follow <a href="https://phpc.social/@phpcs"><code>@phpcs on Mastodon</code></a> or <a href="https://twitter.com/PHP_CodeSniffer"><code>@PHP_CodeSniffer on X</code></a>.</p> <p>If you like what you see, please consider <a href="https://opencollective.com/php_codesniffer">funding the PHP_CodeSniffer project</a>. If you already do so: thank you!</p> </blockquote> </details> <details> <summary>Changelog</summary> <p><em>Sourced from <a href="https://github.com/PHPCSStandards/PHP_CodeSniffer/blob/master/CHANGELOG.md">squizlabs/php_codesniffer's changelog</a>.</em></p> <blockquote> <h2>[3.9.2] - 2024-04-24</h2> <h3>Changed</h3> <ul> <li>The Generic.ControlStructures.DisallowYodaConditions sniff no longer listens for the null coalesce operator. <a href="https://redirect.github.com/PHPCSStandards/PHP_CodeSniffer/issues/458">#458</a> <ul> <li>Thanks to [Rodrigo Primo][<a href="https://github.com/rodrigoprimo"><code>@rodrigoprimo</code></a>] for the patch.</li> </ul> </li> <li>Various housekeeping, including improvements to the tests and documentation. <ul> <li>Thanks to [Dan Wallis][<a href="https://github.com/fredden"><code>@fredden</code></a>], [Rodrigo Primo][<a href="https://github.com/rodrigoprimo"><code>@rodrigoprimo</code></a>] and [Juliette Reinders Folmer][<a href="https://github.com/jrfnl"><code>@jrfnl</code></a>] for their contributions.</li> </ul> </li> </ul> <h3>Fixed</h3> <ul> <li>Fixed bug <a href="https://redirect.github.com/PHPCSStandards/PHP_CodeSniffer/issues/381">#381</a> : Squiz.Commenting.ClosingDeclarationComment could throw the wrong error when the close brace being examined is at the very end of a file. <ul> <li>Thanks to [Rodrigo Primo][<a href="https://github.com/rodrigoprimo"><code>@rodrigoprimo</code></a>] for the patch.</li> </ul> </li> <li>Fixed bug <a href="https://redirect.github.com/PHPCSStandards/PHP_CodeSniffer/issues/385">#385</a> : Generic.CodeAnalysis.JumbledIncrementer improved handling of parse errors/live coding. <ul> <li>Thanks to [Rodrigo Primo][<a href="https://github.com/rodrigoprimo"><code>@rodrigoprimo</code></a>] for the patch.</li> </ul> </li> <li>Fixed bug <a href="https://redirect.github.com/PHPCSStandards/PHP_CodeSniffer/issues/394">#394</a> : Generic.Functions.CallTimePassByReference was not flagging call-time pass-by-reference in anonymous class instantiations <ul> <li>Thanks to [Rodrigo Primo][<a href="https://github.com/rodrigoprimo"><code>@rodrigoprimo</code></a>] for the patch.</li> </ul> </li> <li>Fixed bug <a href="https://redirect.github.com/PHPCSStandards/PHP_CodeSniffer/issues/420">#420</a> : PEAR.Functions.FunctionDeclaration could run into a blocking PHP notice while fixing code containing a parse error. <ul> <li>Thanks to [Juliette Reinders Folmer][<a href="https://github.com/jrfnl"><code>@jrfnl</code></a>] for the patch</li> </ul> </li> <li>Fixed bug <a href="https://redirect.github.com/PHPCSStandards/PHP_CodeSniffer/issues/421">#421</a> : File::getMethodProperties() small performance improvement & more defensive coding. <ul> <li>Thanks to [Juliette Reinders Folmer][<a href="https://github.com/jrfnl"><code>@jrfnl</code></a>] for the patch</li> </ul> </li> <li>Fixed bug <a href="https://redirect.github.com/PHPCSStandards/PHP_CodeSniffer/issues/423">#423</a> : PEAR.WhiteSpace.ScopeClosingBrace would have a fixer conflict with itself when a close tag was preceded by non-empty inline HTML. <ul> <li>Thanks to [Juliette Reinders Folmer][<a href="https://github.com/jrfnl"><code>@jrfnl</code></a>] for the patch</li> </ul> </li> <li>Fixed bug <a href="https://redirect.github.com/PHPCSStandards/PHP_CodeSniffer/issues/424">#424</a> : PSR2.Classes.ClassDeclaration using namespace relative interface names in the extends/implements part of a class declaration would lead to a fixer conflict. <ul> <li>Thanks to [Juliette Reinders Folmer][<a href="https://github.com/jrfnl"><code>@jrfnl</code></a>] for the patch</li> </ul> </li> <li>Fixed bug <a href="https://redirect.github.com/PHPCSStandards/PHP_CodeSniffer/issues/427">#427</a> : Squiz.Operators.OperatorSpacing would have a fixer conflict with itself when an operator was preceeded by a new line and the previous line ended in a comment. <ul> <li>Thanks to [Juliette Reinders Folmer][<a href="https://github.com/jrfnl"><code>@jrfnl</code></a>] for the patch</li> </ul> </li> <li>Fixed bug <a href="https://redirect.github.com/PHPCSStandards/PHP_CodeSniffer/issues/430">#430</a> : Squiz.ControlStructures.ForLoopDeclaration: fixed potential undefined array index notice <ul> <li>Thanks to [Juliette Reinders Folmer][<a href="https://github.com/jrfnl"><code>@jrfnl</code></a>] for the patch</li> </ul> </li> <li>Fixed bug <a href="https://redirect.github.com/PHPCSStandards/PHP_CodeSniffer/issues/431">#431</a> : PSR2.Classes.ClassDeclaration will no longer try to auto-fix multi-line interface implements statements if these are interlaced with comments on their own line. This prevents a potential fixer conflict. <ul> <li>Thanks to [Juliette Reinders Folmer][<a href="https://github.com/jrfnl"><code>@jrfnl</code></a>] for the patch</li> </ul> </li> <li>Fixed bug <a href="https://redirect.github.com/PHPCSStandards/PHP_CodeSniffer/issues/453">#453</a> : Arrow function tokenization was broken when the return type was a stand-alone <code>true</code> or <code>false</code>; or contained <code>true</code> or <code>false</code> as part of a union type. <ul> <li>Thanks to [Juliette Reinders Folmer][<a href="https://github.com/jrfnl"><code>@jrfnl</code></a>] for the patch</li> </ul> </li> </ul> <h3>Other</h3> <ul> <li><a href="https://eslint.org/blog/2024/04/eslint-v9.0.0-released/#flat-config-is-now-the-default-and-has-some-changes">ESLint 9.0</a> has been released and changes the supported configuration file format. The (deprecated) <code>Generic.Debug.ESLint</code> sniff only supports the "old" configuration file formats and when using the sniff to run ESLint, the <code>ESLINT_USE_FLAT_CONFIG=false</code> environment variable will need to be set when using ESLint >= 9.0. For more information, see <a href="https://redirect.github.com/PHPCSStandards/PHP_CodeSniffer/issues/436">#436</a>.</li> </ul> <p><a href="https://redirect.github.com/PHPCSStandards/PHP_CodeSniffer/issues/381">#381</a>: <a href="https://redirect.github.com/PHPCSStandards/PHP_CodeSniffer/pull/381">PHPCSStandards/PHP_CodeSniffer#381</a> <a href="https://redirect.github.com/PHPCSStandards/PHP_CodeSniffer/issues/385">#385</a>: <a href="https://redirect.github.com/PHPCSStandards/PHP_CodeSniffer/pull/385">PHPCSStandards/PHP_CodeSniffer#385</a> <a href="https://redirect.github.com/PHPCSStandards/PHP_CodeSniffer/issues/394">#394</a>: <a href="https://redirect.github.com/PHPCSStandards/PHP_CodeSniffer/pull/394">PHPCSStandards/PHP_CodeSniffer#394</a> <a href="https://redirect.github.com/PHPCSStandards/PHP_CodeSniffer/issues/420">#420</a>: <a href="https://redirect.github.com/PHPCSStandards/PHP_CodeSniffer/pull/420">PHPCSStandards/PHP_CodeSniffer#420</a> <a href="https://redirect.github.com/PHPCSStandards/PHP_CodeSniffer/issues/421">#421</a>: <a href="https://redirect.github.com/PHPCSStandards/PHP_CodeSniffer/pull/421">PHPCSStandards/PHP_CodeSniffer#421</a> <a href="https://redirect.github.com/PHPCSStandards/PHP_CodeSniffer/issues/423">#423</a>: <a href="https://redirect.github.com/PHPCSStandards/PHP_CodeSniffer/pull/423">PHPCSStandards/PHP_CodeSniffer#423</a> <a href="https://redirect.github.com/PHPCSStandards/PHP_CodeSniffer/issues/424">#424</a>: <a href="https://redirect.github.com/PHPCSStandards/PHP_CodeSniffer/pull/424">PHPCSStandards/PHP_CodeSniffer#424</a> <a href="https://redirect.github.com/PHPCSStandards/PHP_CodeSniffer/issues/427">#427</a>: <a href="https://redirect.github.com/PHPCSStandards/PHP_CodeSniffer/pull/427">PHPCSStandards/PHP_CodeSniffer#427</a> <a href="https://redirect.github.com/PHPCSStandards/PHP_CodeSniffer/issues/430">#430</a>: <a href="https://redirect.github.com/PHPCSStandards/PHP_CodeSniffer/pull/430">PHPCSStandards/PHP_CodeSniffer#430</a> <a href="https://redirect.github.com/PHPCSStandards/PHP_CodeSniffer/issues/431">#431</a>: <a href="https://redirect.github.com/PHPCSStandards/PHP_CodeSniffer/pull/431">PHPCSStandards/PHP_CodeSniffer#431</a></p> <!-- raw HTML omitted --> </blockquote> <p>... (truncated)</p> </details> <details> <summary>Commits</summary> <ul> <li><a href="https://github.com/PHPCSStandards/PHP_CodeSniffer/commit/aac1f6f347a5c5ac6bc98ad395007df00990f480"><code>aac1f6f</code></a> Merge pull request <a href="https://redirect.github.com/PHPCSStandards/PHP_CodeSniffer/issues/459">#459</a> from PHPCSStandards/feature/changelog-3.9.2</li> <li><a href="https://github.com/PHPCSStandards/PHP_CodeSniffer/commit/f076b63d43a57ad01eadd6d622d7d5e48b033b61"><code>f076b63</code></a> Changelog for the 3.9.2 release</li> <li><a href="https://github.com/PHPCSStandards/PHP_CodeSniffer/commit/83f38598bafc04c9def87942775b95bfbe08880b"><code>83f3859</code></a> Merge pull request <a href="https://redirect.github.com/PHPCSStandards/PHP_CodeSniffer/issues/460">#460</a> from PHPCSStandards/feature/tests-arrow-functions-vs-...</li> <li><a href="https://github.com/PHPCSStandards/PHP_CodeSniffer/commit/273959e246933750cc36781a6b465d2b777f074d"><code>273959e</code></a> Tokenizer/PHP: add tests for arrow functions with intersection types</li> <li><a href="https://github.com/PHPCSStandards/PHP_CodeSniffer/commit/b0d2d61d782fc61701c06ea6b1745d781b79a398"><code>b0d2d61</code></a> Merge pull request <a href="https://redirect.github.com/PHPCSStandards/PHP_CodeSniffer/issues/453">#453</a> from PHPCSStandards/feature/tokenizer-php-fix-bug-arr...</li> <li><a href="https://github.com/PHPCSStandards/PHP_CodeSniffer/commit/bd6356c3281db766ac61437becfbb14f4c79cfff"><code>bd6356c</code></a> Tokenizer/PHP: arrow function tokenization broken when true/false used in ret...</li> <li><a href="https://github.com/PHPCSStandards/PHP_CodeSniffer/commit/46b883dfc3693aafe3759c9fbcfe0c5b25235326"><code>46b883d</code></a> BackfillFnTokenTest: use data providers when appropriate</li> <li><a href="https://github.com/PHPCSStandards/PHP_CodeSniffer/commit/5f38ce0ab060fcab9441d7ea4b324f7805fae013"><code>5f38ce0</code></a> Merge pull request <a href="https://redirect.github.com/PHPCSStandards/PHP_CodeSniffer/issues/458">#458</a> from rodrigoprimo/disallow-yoda-condition-drop-null-c...</li> <li><a href="https://github.com/PHPCSStandards/PHP_CodeSniffer/commit/e4d374359c9fc857106661b10c5e5107f94849ed"><code>e4d3743</code></a> Generic/DisallowYodaConditions: <code>??</code> should not trigger the sniff</li> <li><a href="https://github.com/PHPCSStandards/PHP_CodeSniffer/commit/8d2363ddb3ef8039d330b4e503ca022b9b7bc298"><code>8d2363d</code></a> Merge pull request <a href="https://redirect.github.com/PHPCSStandards/PHP_CodeSniffer/issues/394">#394</a> from rodrigoprimo/test-coverage-call-time-pass-by-ref...</li> <li>Additional commits viewable in <a href="https://github.com/PHPCSStandards/PHP_CodeSniffer/compare/3.9.1...3.9.2">compare view</a></li> </ul> </details> <br /> [![Dependabot compatibility score](https://dependabot-badges.githubapp.com/badges/compatibility_score?dependency-name=squizlabs/php_codesniffer&package-manager=composer&previous-version=3.9.1&new-version=3.9.2)](https://docs.github.com/en/github/managing-security-vulnerabilities/about-dependabot-security-updates#about-compatibility-scores) Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting `@dependabot rebase`. [//]: # (dependabot-automerge-start) [//]: # (dependabot-automerge-end) --- <details> <summary>Dependabot commands and options</summary> <br /> You can trigger Dependabot actions by commenting on this PR: - `@dependabot rebase` will rebase this PR - `@dependabot recreate` will recreate this PR, overwriting any edits that have been made to it - `@dependabot merge` will merge this PR after your CI passes on it - `@dependabot squash and merge` will squash and merge this PR after your CI passes on it - `@dependabot cancel merge` will cancel a previously requested merge and block automerging - `@dependabot reopen` will reopen this PR if it is closed - `@dependabot close` will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually - `@dependabot show <dependency name> ignore conditions` will show all of the ignore conditions of the specified dependency - `@dependabot ignore this major version` will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself) - `@dependabot ignore this minor version` will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself) - `@dependabot ignore this dependency` will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself) </details>
- Loading branch information