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 dependency automattic/jetpack-codesniffer to v3 #107

Merged
merged 4 commits into from
Feb 8, 2024

Conversation

renovate[bot]
Copy link
Contributor

@renovate renovate bot commented Feb 8, 2024

Mend Renovate

This PR contains the following updates:

Package Change Age Adoption Passing Confidence
automattic/jetpack-codesniffer 2.6.1 -> 3.0.0 age adoption passing confidence

Release Notes

Automattic/jetpack-codesniffer (automattic/jetpack-codesniffer)

v3.0.0

Compare Source

Added
  • Add "Jetpack-NoWP" ruleset. [#​33287]
  • Added rulesets Jetpack-Compat-* to disable PHPCompatibility rules that trigger with testVersion 5.6 but don't apply to various later PHP versions. [#​31711]
  • Add Jetpack-Compat-83 ruleset. [#​34346]
  • Add Jetpack-Compat-NoWP ruleset. [#​33287]
  • Declare requirement of PHP >= 7.4. [#​34192]
  • Enable MediaWiki.AlternativeSyntax.UnicodeEscape sniff. [#​34194]
  • Enable MediaWiki.PHPUnit.MockBoilerplate sniff. [#​34338]
  • Enable Modernize.FunctionCalls.Dirname.Nested sniff that is currently being disabled by WordPress-Extra. [#​34218]
Changed
  • BREAKING: Drop support for PHP 5.6. testVersion should be set to 7.0-. [#​34126]
  • Updated package dependencies. [#​31609]
  • Updated package dependencies. [#​32605]
  • Updated package dependencies. [#​34338]
  • Update to WordPress-Coding-Standards 3.0. [#​32608]
  • Update Jetpack-Compat-* rulesets against PHPCompatibility develop branch. [#​33112]
Removed
  • Exclude new WordPress.Security.EscapeOutput.ExceptionNotEscaped sniff. https://core.trac.wordpress.org/ticket/59282 is the correct way to fix the underlying issue it's trying to avoid, this would cause other problems. [#​32608]
  • Remove MediaWiki.WhiteSpace.SpaceAfterClosure, as Squiz.Functions.MultiLineFunctionDeclaration now catches the same thing and more. [#​32608]

v2.8.0

Compare Source

Added
  • Added MediaWiki.Usage.ForbiddenFunctions rule to use preferred functions
  • Set keywords to have composer require prompt for --dev on installation.
Changed
  • Updated package dependencies.

v2.7.0

Compare Source

Added
  • Add MediaWiki.WhiteSpace.EmptyLinesBetweenUse sniff.
Changed
  • Updated to a newer snapshot of WordPress-Coding-Standards, which adds a number of new sniffs.
Fixed
  • Rolled back a dependency update that caused breaking changes in Jetpack trunk.

Configuration

📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).

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

Rebasing: Whenever PR becomes conflicted, 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 has been generated by Mend Renovate. View repository job log here.

Copy link
Contributor Author

renovate bot commented Feb 8, 2024

⚠ Artifact update problem

Renovate failed to update an artifact related to this branch. You probably do not want to merge this PR as-is.

♻ Renovate will retry this branch, including artifacts, only when one of the following happens:

  • any of the package files in this branch needs updating, or
  • the branch becomes conflicted, or
  • you click the rebase/retry checkbox if found above, or
  • you rename this PR's title to start with "rebase!" to trigger it manually

The artifact failure details are included below:

File name: composer.lock
Command failed: composer update automattic/jetpack-codesniffer:3.0.0 --with-dependencies --ignore-platform-req='ext-*' --ignore-platform-req='lib-*' --no-ansi --no-interaction --no-scripts --no-autoloader --no-plugins
Loading composer repositories with package information
Dependency dealerdirect/phpcodesniffer-composer-installer is also a root requirement. Package has not been listed as an update argument, so keeping locked at old version. Use --with-all-dependencies (-W) to include root dependencies.
Updating dependencies
Your requirements could not be resolved to an installable set of packages.

  Problem 1
    - Root composer.json requires automattic/jetpack-codesniffer 3.0.0 -> satisfiable by automattic/jetpack-codesniffer[v3.0.0].
    - automattic/jetpack-codesniffer v3.0.0 requires dealerdirect/phpcodesniffer-composer-installer ^1.0 -> found dealerdirect/phpcodesniffer-composer-installer[v1.0.0] but it conflicts with your root composer.json require (0.7.2).

Use the option --with-all-dependencies (-W) to allow upgrades, downgrades and removals for packages currently locked to specific versions.

Copy link
Contributor Author

renovate bot commented Feb 8, 2024

Edited/Blocked Notification

Renovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR.

You can manually request rebase by checking the rebase/retry box above.

Warning: custom changes will be lost.

@jeherve jeherve merged commit 0b552c7 into master Feb 8, 2024
6 checks passed
@jeherve jeherve deleted the renovate/automattic-jetpack-codesniffer-3.x branch February 8, 2024 08:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

1 participant