Skip to content

Update dependency squizlabs/php_codesniffer to ^3.11.2 #961

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

Merged
merged 1 commit into from
Dec 25, 2024

Conversation

renovate[bot]
Copy link
Contributor

@renovate renovate bot commented Nov 12, 2024

This PR contains the following updates:

Package Change Age Adoption Passing Confidence
squizlabs/php_codesniffer ^3.10.3 -> ^3.11.2 age adoption passing confidence

Release Notes

PHPCSStandards/PHP_CodeSniffer (squizlabs/php_codesniffer)

v3.11.2

Compare Source

Changed
  • Generators/HTML + Markdown: the output will now be empty (no page header/footer) when there are no docs to display. #​687
    • This is in line with the Text Generator which already didn't produce output if there are no docs.
    • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch.
  • Generators/HTML: only display a Table of Contents when there is more than one sniff with documentation. #​697
    • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch.
  • Generators/HTML: improved handling of line breaks in <standard> blocks. #​723
    • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch.
  • Generators/Markdown: improved compatibility with the variety of available markdown parsers. #​722
    • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch.
  • Generators/Markdown: improved handling of line breaks in <standard> blocks. #​737
    • This prevents additional paragraphs from being displayed as code blocks.
    • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch.
  • Generic.NamingConventions.UpperCaseConstantName: the exact token containing the non-uppercase constant name will now be identified with more accuracy. #​665
  • Generic.Functions.OpeningFunctionBraceKernighanRitchie: minor improvement to the error message wording. #​736
    • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch.
  • Various housekeeping, including improvements to the tests and documentation.
Fixed
  • Fixed bug #​527 : Squiz.Arrays.ArrayDeclaration: short lists within a foreach condition should be ignored.
  • Fixed bug #​665 : Generic.NamingConventions.UpperCaseConstantName: false positives and false negatives when code uses unconventional spacing and comments when calling define().
  • Fixed bug #​665 : Generic.NamingConventions.UpperCaseConstantName: false positive when a constant named DEFINE is encountered.
  • Fixed bug #​665 : Generic.NamingConventions.UpperCaseConstantName: false positive for attribute class called define.
  • Fixed bug #​665 : Generic.NamingConventions.UpperCaseConstantName: false positive when handling the instantiation of a class named define.
  • Fixed bug #​688 : Generators/Markdown could leave error_reporting in an incorrect state.
    • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch.
  • Fixed bug #​698 : Generators/Markdown : link in the documentation footer would not parse as a link.
    • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch.
  • Fixed bug #​738 : Generators/Text: stray blank lines after code sample titles.
    • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch.
  • Fixed bug #​739 : Generators/HTML + Markdown: multi-space whitespace within a code sample title was folded into a single space.
    • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch.

v3.11.1

Compare Source

Changed
  • Output from the --generator=... feature will respect the OS-expected EOL char in more places. #​671
    • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch.
  • Various housekeeping, including improvements to the tests and documentation.
    • Thanks to [Bartosz Dziewoński][@​MatmaRex] and [Juliette Reinders Folmer][@​jrfnl] for their contributions.
Fixed
  • Fixed bug #​674 : Generic.WhiteSpace.HereNowdocIdentifierSpacing broken XML documentation
    • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch.
  • Fixed bug #​675 : InvalidArgumentException when a ruleset includes a sniff by file name and the included sniff does not comply with the PHPCS naming conventions.
    • Notwithstanding this fix, it is strongly recommended to ensure custom sniff classes comply with the PHPCS naming conventions.
    • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch.

v3.11.0

Compare Source

Added
  • Runtime support for PHP 8.4. All known PHP 8.4 deprecation notices have been fixed.
    • Syntax support for new PHP 8.4 features will follow in a future release.
    • If you find any PHP 8.4 deprecation notices which were missed, please report them.
    • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patches.
  • Tokenizer support for PHP 8.3 "yield from" expressions with a comment between the keywords. #​529, #​647
    • Sniffs explicitly handling T_YIELD_FROM tokens may need updating. The PR description contains example code for use by sniff developers.
    • Additionally, the following sniff has been updated to support "yield from" expressions with comments:
      • Generic.WhiteSpace.LanguageConstructSpacing
    • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch.
  • New Generic.WhiteSpace.HereNowdocIdentifierSpacing sniff. #​586, #​637
    • Forbid whitespace between the <<< and the identifier string in heredoc/nowdoc start tokens.
    • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch.
  • New Generic.Strings.UnnecessaryHeredoc sniff. #​633
    • Warns about heredocs without interpolation or expressions in the body text and can auto-fix these to nowdocs.
    • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch.
  • Documentation for the following sniffs:
    • Generic.Arrays.ArrayIndent
    • Squiz.PHP.Heredoc
    • Thanks to [Rodrigo Primo][@​rodrigoprimo] and [Juliette Reinders Folmer][@​jrfnl] for the patches.
Changed
  • The Common::getSniffCode() method will now throw an InvalidArgumentException exception if an invalid $sniffClass is passed. #​524, #​625
    • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch.
  • Documentation generated using the --generator=... feature will now always be presented in natural order based on the sniff name(s). #​668
    • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch.
  • Minor improvements to the display of runtime information. #​658
    • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch.
  • Squiz.Commenting.PostStatementComment: trailing annotations in PHP files will now be reported under a separate, non-auto-fixable error code AnnotationFound. #​560, #​627
    • This prevents (tooling related) annotations from taking on a different meaning when moved by the fixer.
    • The separate error code also allows for selectively excluding it to prevent the sniff from triggering on trailing annotations, while still forbidding other trailing comments.
    • Thanks to [Rodrigo Primo][@​rodrigoprimo] for the patch.
  • Squiz.ControlStructures.ForEachLoopDeclaration: the SpacingAfterOpen error code has been replaced by the SpaceAfterOpen error code. The latter is a pre-existing code. The former appears to have been a typo. #​582
  • The following sniff(s) have received efficiency improvements:
    • Generic.Classes.DuplicateClassName
    • Generic.NamingConventions.ConstructorName
    • Thanks to [Rodrigo Primo][@​rodrigoprimo] and [Juliette Reinders Folmer][@​jrfnl] for the patches.
  • Various housekeeping, including improvements to the tests and documentation.
Fixed
  • Fixed bug #​3808 : Generic.WhiteSpace.ScopeIndent would throw false positive for tab indented multi-token yield from expression.
    • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch
  • Fixed bug #​630 : The tokenizer could inadvertently transform "normal" parentheses to DNF parentheses, when a function call was preceded by a switch-case / alternative syntax control structure colon.
    • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch
  • Fixed bug #​645 : On PHP 5.4, if yield was used as the declaration name for a function declared to return by reference, the function name would incorrectly be tokenized as T_YIELD instead of T_STRING.
    • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch
  • Fixed bug #​647 : Tokenizer not applying tab replacement in single token "yield from" keywords.
    • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch
  • Fixed bug #​647 : Generic.WhiteSpace.DisallowSpaceIndent did not flag space indentation in multi-line yield from.
    • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch
  • Fixed bug #​647 : Generic.WhiteSpace.DisallowTabIndent did not flag tabs inside yield from.
    • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch
  • Fixed bug #​652 : Generic.NamingConventions.ConstructorName: false positives for PHP-4 style calls to PHP-4 style parent constructor when a method with the same name as the parent class was called on another class.
    • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch
  • Fixed bug #​652 : Generic.NamingConventions.ConstructorName: false negatives for PHP-4 style calls to parent constructor for function calls with whitespace and comments in unconventional places.
    • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch
  • Fixed bug #​653 : Generic.Classes.DuplicateClassName : the sniff did not skip namespace keywords used as operators, which could lead to false positives.
    • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch
  • Fixed bug #​653 : Generic.Classes.DuplicateClassName : sniff going into an infinite loop during live coding.
    • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch
  • Fixed bug #​653 : Generic.Classes.DuplicateClassName : false positives/negatives when a namespace declaration contained whitespace or comments in unconventional places.
    • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch
  • Fixed bug #​653 : Generic.Classes.DuplicateClassName : namespace for a file going in/out of PHP was not remembered/applied correctly.
    • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch

Configuration

📅 Schedule: Branch creation - "* 6-22/3 * * 1-5,* 12-23/3 * * 0,6" in timezone Europe/Amsterdam, Automerge - At any time (no schedule defined).

🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.

Rebasing: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.

🔕 Ignore: Close this PR and you won't be reminded about this update again.


  • If you want to rebase/retry this PR, check this box

This PR was generated by Mend Renovate. View the repository job log.

@renovate renovate bot requested a review from WyriHaximus as a code owner November 12, 2024 11:02
@renovate renovate bot added Dependencies 📦 Pull requests that update a dependency file PHP 🐘 Hypertext Pre Processor labels Nov 12, 2024
Copy link

github-actions bot commented Nov 12, 2024

🏰 Composer Production Dependency changes 🏰

Prod Packages Operation Base Target Link
squizlabs/php_codesniffer Upgraded 3.10.3 3.11.2 Compare

@github-actions github-actions bot added this to the 6.1.0 milestone Nov 12, 2024
@renovate renovate bot force-pushed the renovate/squizlabs-php_codesniffer-3.x branch from 016e8ae to a84c159 Compare November 16, 2024 13:38
@renovate renovate bot changed the title Update dependency squizlabs/php_codesniffer to ^3.11.0 Update dependency squizlabs/php_codesniffer to ^3.11.1 Nov 16, 2024
@renovate renovate bot force-pushed the renovate/squizlabs-php_codesniffer-3.x branch from a84c159 to 55d7bd4 Compare November 26, 2024 07:26
@renovate renovate bot force-pushed the renovate/squizlabs-php_codesniffer-3.x branch 2 times, most recently from 2aa9ddc to 6727aae Compare December 11, 2024 18:14
@renovate renovate bot changed the title Update dependency squizlabs/php_codesniffer to ^3.11.1 Update dependency squizlabs/php_codesniffer to ^3.11.2 Dec 11, 2024
@renovate renovate bot force-pushed the renovate/squizlabs-php_codesniffer-3.x branch from 6727aae to 89ac062 Compare December 12, 2024 13:51
@renovate renovate bot force-pushed the renovate/squizlabs-php_codesniffer-3.x branch from 89ac062 to 3652284 Compare December 23, 2024 16:35
@WyriHaximus WyriHaximus modified the milestones: 6.1.0, 7.0.0 Dec 24, 2024
@renovate renovate bot force-pushed the renovate/squizlabs-php_codesniffer-3.x branch from 3652284 to 9c663e7 Compare December 24, 2024 10:41
@renovate renovate bot force-pushed the renovate/squizlabs-php_codesniffer-3.x branch from 9c663e7 to 8093634 Compare December 24, 2024 10:43
@WyriHaximus WyriHaximus merged commit 2ad7bd9 into master Dec 25, 2024
172 of 174 checks passed
@WyriHaximus WyriHaximus deleted the renovate/squizlabs-php_codesniffer-3.x branch December 25, 2024 12:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Dependencies 📦 Pull requests that update a dependency file JSON 👨‍💼 PHP 🐘 Hypertext Pre Processor
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant