Skip to content

Conversation

@dependabot-preview
Copy link
Contributor

@dependabot-preview dependabot-preview bot commented Sep 4, 2018

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

Release notes

Sourced from squizlabs/php_codesniffer's releases.

3.3.1

HHVM Support Dropped

Support for HHVM has been dropped due to recent unfixed bugs and HHVM's refocus on Hack only. Thanks to Walt Sorensen and Juliette Reinders Folmer for helping to remove all HHVM exceptions from the core.

Other Changes

  • The full report (the default report) now has improved word wrapping for multi-line messages and sniff codes
    • Thanks to Juliette Reinders Folmer for the patch
  • The summary report now sorts files based on their directory location instead of just a basic string sort
    • Thanks to Juliette Reinders Folmer for the patch
  • The source report now orders error codes by name when they have the same number of errors
    • Thanks to Juliette Reinders Folmer for the patch
  • The junit report no longer generates validation errors with the Jenkins xUnit plugin
    • Thanks to Nikolay Geo for the patch
  • Generic.Commenting.DocComment no longer generates the SpacingBeforeTags error if tags are the first content in the docblock
    • The sniff will still generate a MissingShort error if there is no short comment
    • This allows the MissingShort error to be suppressed in a ruleset to make short descriptions optional
  • Generic.Functions.FunctionCallArgumentSpacing now properly fixes multi-line function calls with leading commas
    • Previously, newlines between function arguments would be removed
    • Thanks to Juliette Reinders Folmer for the patch
  • Generic.PHP.Syntax will now use PHP_BINARY instead of trying to discover the executable path
    • This ensures that the sniff will always syntax check files using the PHP version that PHPCS is running under
    • Setting the php_path config var will still override this value as normal
    • Thanks to Willem Stuursma-Ruwen for the patch
  • PSR2.Namespaces.UseDeclaration now supports commas at the end of group use declarations
    • Also improves checking and fixing for use statements containing parse errors
    • Thanks to Juliette Reinders Folmer for the patch
  • Squiz.Arrays.ArrayDeclaration no longer removes the array opening brace while fixing
    • This could occur when the opening brace was on a new line and the first array key directly followed
    • This change also stops the KeyNotAligned error message being incorrectly reported in these cases
  • Squiz.Arrays.ArrayDeclaration no longer tries to change multi-line arrays to single line when they contain comments
    • Fixes a conflict between this sniff and some indentation sniffs
  • Squiz.Classes.ClassDeclaration no longer enforces spacing rules when a class is followed by a function
    • Fixes a conflict between this sniff and the Squiz.WhiteSpace.FunctionSpacing sniff
  • The Squiz.Classes.ValidClassName.NotCamelCaps message now references PascalCase instead of CamelCase
    • The CamelCase class name metric produced by the sniff has been changed to PascalCase class name
    • This reflects the fact that the class name check is actually a Pascal Case check and not really Camel Case
    • Thanks to Tom H Anderson for the patch
  • Squiz.Commenting.InlineComment no longer enforces spacing rules when an inline comment is followed by a docblock
    • Fixes a conflict between this sniff and the Squiz.WhiteSpace.FunctionSpacing sniff
  • Squiz.WhiteSpace.OperatorSpacing no longer tries to fix operator spacing if the next content is a comment on a new line
    • Fixes a conflict between this sniff and the Squiz.Commenting.PostStatementComment sniff
    • Also stops PHPCS annotations from being moved to a different line, potentially changing their meaning
    • Thanks to Juliette Reinders Folmer for the patch
  • Squiz.WhiteSpace.FunctionSpacing no longer checks spacing of functions at the top of an embedded PHP block
    • Fixes a conflict between this sniff and the Squiz.PHP.EmbeddedPHP sniff
    • Thanks to Juliette Reinders Folmer for the patch
  • Squiz.WhiteSpace.MemberVarSpacing no longer checks spacing before member vars that come directly after methods
    • Fixes a conflict between this sniff and the Squiz.WhiteSpace.FunctionSpacing sniff
  • Squiz.WhiteSpace.SuperfluousWhitespace now recognizes unicode whitespace at the start and end of a file
    • Thanks to 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.


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.

Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot ignore this [patch|minor|major] version will close this PR and stop Dependabot creating any more for this minor/major 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)
  • @dependabot use these labels will set the current labels as the default for future PRs for this repo and language
  • @dependabot use these reviewers will set the current reviewers as the default for future PRs for this repo and language
  • @dependabot use these assignees will set the current assignees as the default for future PRs for this repo and language
  • @dependabot use this milestone will set the current milestone as the default for future PRs for this repo and language
  • @dependabot badge me will comment on this PR with code to add a "Dependabot enabled" badge to your readme

Additionally, you can set the following in your Dependabot dashboard:

  • Update frequency (including time of day and day of week)
  • Automerge options (never/patch/minor, and dev/runtime dependencies)
  • Pull request limits (per update run and/or open at any time)
  • Out-of-range updates (receive only lockfile updates, if desired)
  • Security updates (receive only security updates, if desired)

Finally, you can contact us by mentioning @dependabot.


⚠️ Dependabot is rebasing this PR ⚠️

This PR will be updated for you in a moment. If you make any changes yourself then they will take precedence over the rebase (which will be abandoned).

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.3.1)

Signed-off-by: dependabot[bot] <support@dependabot.com>
@dependabot-preview dependabot-preview bot added the dependencies Pull requests that update a dependency file label Sep 4, 2018
@hyperized hyperized merged commit 0c00ddf into master Sep 4, 2018
@dependabot-preview dependabot-preview bot deleted the dependabot/composer/squizlabs/php_codesniffer-3.3.1 branch September 4, 2018 13:09
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.

3 participants