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

Update squizlabs/php_codesniffer requirement to ^3.2 #2

Merged

Conversation

dependabot-preview[bot]
Copy link
Contributor

Updates the requirements on squizlabs/php_codesniffer to permit the latest version.

Release notes

Sourced from squizlabs/php_codesniffer's releases.

3.2.3

  • The new phpcs: comment syntax can now be prefixed with an at symbol ( [**phpcs**](https://github.com/phpcs): )
    • This restores the behaviour of the previous syntax where these comments are ignored by doc generators
  • The current PHP version ID is now used to generate cache files
    • This ensures that only cache files generated by the current PHP version are selected
    • This change fixes caching issues when using sniffs that produce errors based on the current PHP version
  • A new Tokens::$phpcsCommentTokens array is now available for sniff developers to detect phpcs: comment syntax
    • Thanks to Juliette Reinders Folmer for the patch
  • Error message codes generated by Generic.CodeAnalysis.EmptyStatement are no longer all-uppercase
    • For example Generic.CodeAnalysis.EmptyStatement.DetectedCATCH becomes Generic.CodeAnalysis.EmptyStatement.DetectedCatch
  • The PEAR.Commenting.FunctionComment.Missing error message now includes the name of the function
    • Thanks to Yorman Arias for the patch
  • The PEAR.Commenting.ClassComment.Missing and Squiz.Commenting.ClassComment.Missing error messages now include the name of the class
    • Thanks to Yorman Arias for the patch
  • PEAR.Functions.FunctionCallSignature now only forces alignment at a specific tab stop while fixing
    • It was enforcing this during checking, but this meant invalid errors if the OpeningIndent message was being muted
    • This fixes incorrect errors when using the PSR2 standard with some code blocks
  • Generic.Files.LineLength now ignores lines that only contain phpcs: annotation comments
    • Thanks to Juliette Reinders Folmer for the patch
  • Generic.Formatting.MultipleStatementAlignment now skips over arrays containing comments
    • Thanks to Juliette Reinders Folmer for the patch
  • Generic.PHP.Syntax now forces display_errors to ON when linting
    • Thanks to Raúl Arellano for the patch
  • PSR2.Namespaces.UseDeclaration has improved syntax error handling and closure detection
    • Thanks to Juliette Reinders Folmer for the patch
  • Squiz.PHP.CommentedOutCode now has improved comment block detection for improved accuracy
    • Thanks to Juliette Reinders Folmer for the patch
  • Squiz.PHP.NonExecutableCode could fatal error while fixing file with syntax error
  • Squiz.PHP.NonExecutableCode now detects unreachable code after a goto statement
    • Thanks to Juliette Reinders Folmer for the patch
  • Squiz.WhiteSpace.LanguageConstructSpacing has improved syntax error handling while fixing
    • Thanks to Juliette Reinders Folmer for the patch
  • Improved phpcs: annotation syntax handling for a number of sniffs
    • Thanks to Juliette Reinders Folmer for the patch
  • Improved auto-fixing of files with incomplete comment blocks for various commenting sniffs
    • Thanks to Juliette Reinders Folmer for the patch
  • Fixed test suite compatibility with PHPUnit 7
  • Fixed bug #1793 : PSR2 forcing exact indent for function call opening statements
  • Fixed bug #1803 : Squiz.WhiteSpace.ScopeKeywordSpacing removes member var name while fixing if no space after scope keyword
  • Fixed bug #1817 : Blank line not enforced after control structure if comment on same line as closing brace
  • Fixed bug #1827 : A phpcs:enable comment is not tokenized correctly if it is outside a phpcs:disable block
    • Thanks to Juliette Reinders Folmer for the patch
  • Fixed bug #1828 : Squiz.WhiteSpace.SuperfluousWhiteSpace ignoreBlankLines property ignores whitespace after single line comments
    • Thanks to Juliette Reinders Folmer for the patch
  • Fixed bug #1840 : When a comment has too many asterisks, phpcbf gives FAILED TO FIX error
  • Fixed bug #1867 : Cant use phpcs:ignore where the next line is HTML
  • Fixed bug #1870 : Invalid warning in multiple assignments alignment with closure or anon class
  • Fixed bug #1890 : Incorrect Squiz.WhiteSpace.ControlStructureSpacing.NoLineAfterClose error between catch and finally statements
  • Fixed bug #1891 : Comment on last USE statement causes false positive for PSR2.Namespaces.UseDeclaration.SpaceAfterLastUse
    • Thanks to Matt Coleman, Daniel Hensby, and Juliette Reinders Folmer for the patch
      ... (truncated)
Commits

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.

If you'd like to skip this version, you can just close this PR. If you have any feedback just mention @dependabot in the comments below.


Note: This repo was added to Dependabot recently, so you'll receive a maximum of 5 PRs for your first few update runs. Once an update run creates fewer than 5 PRs we'll remove that limit.

You can always request more updates by clicking Bump now in your Dependabot dashboard.

Updates the requirements on [squizlabs/php_codesniffer](https://github.com/squizlabs/PHP_CodeSniffer) to permit the latest version.
- [Release notes](https://github.com/squizlabs/PHP_CodeSniffer/releases)
- [Commits](https://github.com/squizlabs/PHP_CodeSniffer/commits/3.2.3)

Signed-off-by: dependabot[bot] <support@dependabot.com>
@weirdan weirdan merged commit 3dc5c52 into master Mar 17, 2018
@dependabot-preview dependabot-preview bot deleted the dependabot/composer/squizlabs/php_codesniffer-tw-3.2 branch March 17, 2018 19:56
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.

2 participants