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

Bump squizlabs/php_codesniffer from 3.8.0 to 3.8.1 #131

Merged

Conversation

dependabot[bot]
Copy link
Contributor

@dependabot dependabot bot commented on behalf of github Jan 12, 2024

Bumps squizlabs/php_codesniffer from 3.8.0 to 3.8.1.

Release notes

Sourced from squizlabs/php_codesniffer's releases.

3.8.1 - 2024-01-11

Added

  • Documentation has been added for the following sniffs:
    • Generic.CodeAnalysis.EmptyPHPStatement
    • Generic.Formatting.SpaceBeforeCast
    • Generic.PHP.Syntax
    • Generic.WhiteSpace.LanguageConstructSpacing
    • PSR12.Classes.ClosingBrace
    • PSR12.Classes.OpeningBraceSpace
    • PSR12.ControlStructures.BooleanOperatorPlacement
    • PSR12.Files.OpenTag
    • Thanks to [Rodrigo Primo][@​rodrigoprimo] and [Denis Žoljom][@​dingo-d] for the patches

Changed

  • GitHub releases will now always only contain unversioned versions of the release assets (PHARS + asc files). See #205 for context.
    • Thanks to [Shivam Mathur][@​shivammathur] for opening a discussion about this
  • Various housekeeping, includes improvements to the tests and documentation

Fixed

  • Fixed bug #124 : Report Full : avoid unnecessarily wrapping lines when -s is used
  • Fixed bug #124 : Report Full : fix incorrect bolding of pipes when -s is used and messages wraps
  • Fixed bug #150 : Squiz.WhiteSpace.KeywordSpacing : prevent a PHP notice when run during live coding
    • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch
  • Fixed bug #154 : Report Full : delimiter line calculation could go wonky on wide screens when a report contains multi-line messages
    • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch
  • Fixed bug #178 : Squiz.Commenting.VariableComment : docblocks were incorrectly being flagged as missing when a property declaration used PHP native union/intersection type declarations
    • Thanks to [Ferdinand Kuhl][@​fcool] for the patch
  • Fixed bug #211 : Squiz.Commenting.VariableComment : docblocks were incorrectly being flagged as missing when a property declaration used PHP 8.2+ stand-alone true/false/null type declarations
    • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch
  • Fixed bug #211 : Squiz.Commenting.VariableComment : docblocks were incorrectly being flagged as missing when a property declaration used PHP native parent, self or a namespace relative class name type declaration
    • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch
  • Fixed bug #226 : Generic.CodeAnalysis.ForLoopShouldBeWhileLoop : prevent a potential PHP 8.3 deprecation notice during live coding

New Contributors

The PHP_CodeSniffer project is happy to welcome the following new contributors: @​anomiex, @​dingo-d, @​fcool, @​lucc, @​rodrigoprimo

Statistics

Closed: 4 issues Merged: 63 pull requests

If you like what you see, please consider funding the PHP_CodeSniffer project. If you already do so: thank you!

... (truncated)

Changelog

Sourced from squizlabs/php_codesniffer's changelog.

[3.8.1] - 2024-01-11

Added

  • Documentation has been added for the following sniffs:
    • Generic.CodeAnalysis.EmptyPHPStatement
    • Generic.Formatting.SpaceBeforeCast
    • Generic.PHP.Syntax
    • Generic.WhiteSpace.LanguageConstructSpacing
    • PSR12.Classes.ClosingBrace
    • PSR12.Classes.OpeningBraceSpace
    • PSR12.ControlStructures.BooleanOperatorPlacement
    • PSR12.Files.OpenTag
    • Thanks to [Rodrigo Primo][@​rodrigoprimo] and [Denis Žoljom][@​dingo-d] for the patches

Changed

  • GitHub releases will now always only contain unversioned release assets (PHARS + asc files) (same as it previously was in the squizlabs repo). See #205 for context.
    • Thanks to [Shivam Mathur][@​shivammathur] for opening a discussion about this
  • Various housekeeping, includes improvements to the tests and documentation

Fixed

  • Fixed bug #124 : Report Full : avoid unnecessarily wrapping lines when -s is used
  • Fixed bug #124 : Report Full : fix incorrect bolding of pipes when -s is used and messages wraps
  • Fixed bug #150 : Squiz.WhiteSpace.KeywordSpacing : prevent a PHP notice when run during live coding
    • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch
  • Fixed bug #154 : Report Full : delimiter line calculation could go wonky on wide screens when a report contains multi-line messages
    • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch
  • Fixed bug #178 : Squiz.Commenting.VariableComment : docblocks were incorrectly being flagged as missing when a property declaration used PHP native union/intersection type declarations
    • Thanks to [Ferdinand Kuhl][@​fcool] for the patch
  • Fixed bug #211 : Squiz.Commenting.VariableComment : docblocks were incorrectly being flagged as missing when a property declaration used PHP 8.2+ stand-alone true/false/null type declarations
    • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch
  • Fixed bug #211 : Squiz.Commenting.VariableComment : docblocks were incorrectly being flagged as missing when a property declaration used PHP native parent, self or a namespace relative class name type declaration
    • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch
  • Fixed bug #226 : Generic.CodeAnalysis.ForLoopShouldBeWhileLoop : prevent a potential PHP 8.3 deprecation notice during live coding

#124: PHPCSStandards/PHP_CodeSniffer#124 #150: PHPCSStandards/PHP_CodeSniffer#150 #154: PHPCSStandards/PHP_CodeSniffer#154 #178: PHPCSStandards/PHP_CodeSniffer#178 #205: PHPCSStandards/PHP_CodeSniffer#205 #211: PHPCSStandards/PHP_CodeSniffer#211 #226: PHPCSStandards/PHP_CodeSniffer#226

Commits
  • 14f5fff Merge pull request #251 from PHPCSStandards/feature/changelog-3.8.1
  • 8f5f986 Changelog for the 3.8.1 release
  • 65270eb Tests/SuggestTypeTest: minor docblock fix
  • 364cc72 Merge pull request #246 from PHPCSStandards/feature/util-common-suggesttype-a...
  • 63ab52c Util/Common: add tests for the suggestType() method
  • f58a158 Merge pull request #245 from PHPCSStandards/feature/ci-prevent-issues-with-co...
  • 7341d76 GH Actions: warm PHPUnit cache on PHPUnit 9.3+
  • cbb5548 Merge pull request #211 from PHPCSStandards/feature/squiz-variablecomment-all...
  • b04f0a6 Squiz/VariableComment: allow for various other missing types
  • 7725dec PHP 8.2 | Squiz/VariableComment: allow for stand-alone null/true/false types
  • Additional commits viewable in compare view

Dependabot compatibility score

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 commands and options

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)

Bumps [squizlabs/php_codesniffer](https://github.com/PHPCSStandards/PHP_CodeSniffer) from 3.8.0 to 3.8.1.
- [Release notes](https://github.com/PHPCSStandards/PHP_CodeSniffer/releases)
- [Changelog](https://github.com/PHPCSStandards/PHP_CodeSniffer/blob/master/CHANGELOG.md)
- [Commits](PHPCSStandards/PHP_CodeSniffer@3.8.0...3.8.1)

---
updated-dependencies:
- dependency-name: squizlabs/php_codesniffer
  dependency-type: direct:development
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
@dependabot dependabot bot added the dependencies Pull requests that update a dependency file label Jan 12, 2024
@coveralls
Copy link

Coverage Status

coverage: 100.0%. remained the same
when pulling 713e36e on dependabot/composer/squizlabs/php_codesniffer-3.8.1
into 454d386 on master.

@renekorss renekorss merged commit 8d4ab6c into master Jan 12, 2024
11 checks passed
@dependabot dependabot bot deleted the dependabot/composer/squizlabs/php_codesniffer-3.8.1 branch January 12, 2024 11:08
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants