quarkus-qe / quarkus-test-suite

16 stars 34 forks source link

Bump com.puppycrawl.tools:checkstyle from 10.18.2 to 10.20.0 #2157

Closed dependabot[bot] closed 3 weeks ago

dependabot[bot] commented 4 weeks ago

Bumps com.puppycrawl.tools:checkstyle from 10.18.2 to 10.20.0.

Release notes

Sourced from com.puppycrawl.tools:checkstyle's releases.

checkstyle-10.20.0

Checkstyle 10.20.0 - https://checkstyle.org/releasenotes.html#Release_10.20.0

Breaking backward compatibility:

#15687 - JavadocMethodCheck: removed unnecessary tokens from acceptable

New:

#14424 - HideUtilityClassConstructor - Add option to skip validation based on list of annotations

Bug fixes:

#15831 - google_checks.xml not allowing eol left curly for switch statement with lambda-like construct

checkstyle-10.19.0

Checkstyle 10.19.0 - https://checkstyle.org/releasenotes.html#Release_10.19.0

New:

#9540 - WhitespaceAround: new property allowEmptySwitchBlockStatements #15263 - UnnecessaryParenthesesCheck does not flag unnecessary parentheses in conditional expression

Bug fixes:

#15664 - false-negative in google_checks.xml for not being able to detect requirement of K & R style for FINALLY #15769 - google_checks.xml: remove xpath suppression and false-positive indentation violations for block codes #15685 - JavadocParagraph does not work when paragraphs have their corresponding closing tag #15324 - Enforce preceding line break for opening braces of a case/default under switch in google_checks.xml #15733 - JavadocParagraph: report violation with column #15503 - JavadocParagraph: violate preceding P tag before block-level HTML tags #15716 - google_checks.xml: JavadocParagraph should have allowNewlineParagraph as false

... (truncated)

Commits
  • 41e15b3 [maven-release-plugin] prepare release checkstyle-10.20.0
  • 719ae40 doc: release notes for 10.20.0
  • 4c67922 Issue #15831: enabled allowEmptySwitchBlockStatements property of WhitespaceA...
  • 67b98ab Issue #14814: refactor checkline into iterative method
  • 9db3909 Issue #14814: refactor findmatch into iteration method
  • 853e2ba Issue #13345: Enable examples tests for ExplicitInitializationCheck
  • 8e8df58 Issue #13345: Enable examples tests for CovariantEqualsCheck
  • ca693c7 Issue #15829: Added test class for ConstructorsDeclarationGrouping
  • c256c10 Issue #6207: Added XPath regression test for ClassTypeParameterName
  • 42cf0ad Issue #15456: Specify violation messages for ArrayTrailingComma
  • Additional commits viewable in compare view


Most Recent Ignore Conditions Applied to This Pull Request | Dependency Name | Ignore Conditions | | --- | --- | | com.puppycrawl.tools:checkstyle | [< 10.13, > 10.12.0] |

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 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)
jedla97 commented 3 weeks ago

@dependabot rebase

github-actions[bot] commented 3 weeks ago

Following jobs contain at least one flaky test:

Run summary: https://github.com/quarkus-qe/quarkus-test-suite/actions/runs/11681583679?pr=2157

Flaky tests:


io.quarkus.ts.sqldb.panacheflyway.dbpool.AgroalPoolTest.idleTimeoutTest

org.opentest4j.AssertionFailedError: agroalCheckIdleTimeout: Expected 1 active connections ==> expected: <1> but was: <3>
    at org.junit.jupiter.api.AssertionFailureBuilder.build(AssertionFailureBuilder.java:151)
    at org.junit.jupiter.api.AssertionFailureBuilder.buildAndThrow(AssertionFailureBuilder.java:132)
    at org.junit.jupiter.api.AssertEquals.failNotEqual(AssertEquals.java:197)
    at org.junit.jupiter.api.AssertEquals.assertEquals(AssertEquals.java:182)
    at org.junit.jupiter.api.Assertions.assertEquals(Assertions.java:672)
    at io.quarkus.ts.sqldb.panacheflyway.dbpool.AgroalPoolTest.idleTimeoutTest(AgroalPoolTest.java:102)
    at java.base/java.lang.reflect.Method.invoke(Method.java:569)
    at io.quarkus.test.junit.QuarkusTestExtension.runExtensionMethod(QuarkusTestExtension.java:971)
    at io.quarkus.test.junit.QuarkusTestExtension.interceptTestMethod(QuarkusTestExtension.java:821)
    at java.base/java.util.ArrayList.forEach(ArrayList.java:1511)
    at java.base/java.util.ArrayList.forEach(ArrayList.java:1511)