Skip to content

Commit

Permalink
ORC-1763: Upgrade checkstyle to 10.18.0
Browse files Browse the repository at this point in the history
Bumps [com.puppycrawl.tools:checkstyle](https://github.com/checkstyle/checkstyle) from 10.17.0 to 10.18.0.
<details>
<summary>Release notes</summary>
<p><em>Sourced from <a href="https://github.com/checkstyle/checkstyle/releases">com.puppycrawl.tools:checkstyle's releases</a>.</em></p>
<blockquote>
<h2>checkstyle-10.18.0</h2>
<p>Checkstyle 10.18.0 - <a href="https://checkstyle.org/releasenotes.html#Release_10.18.0">https://checkstyle.org/releasenotes.html#Release_10.18.0</a></p>
<p>Breaking backward compatibility:</p>
<p><a href="https://redirect.github.com/checkstyle/checkstyle/issues/15159">#15159</a> - RedundantModifer Should Violate Final Modifer on Unnamed Variables if jdkVersion GTE 22
<a href="https://redirect.github.com/checkstyle/checkstyle/issues/14805">#14805</a> - Remove Support for String Template Syntax
<a href="https://redirect.github.com/checkstyle/checkstyle/issues/15082">#15082</a> - LITERAL_WHEN and LAND tokens in switch statements should have EXPR child</p>
<p>New:</p>
<p><a href="https://redirect.github.com/checkstyle/checkstyle/issues/10798">#10798</a> - RedundantModifierCheck: New property 'jdkVersion' to violate strictfp as redundant modifier if version LTE 17
<a href="https://redirect.github.com/checkstyle/checkstyle/issues/15062">#15062</a> - Add Check Support for Java 21 Unnamed Variables <code>&amp;</code> Patterns Syntax: SuppressWarnings
<a href="https://redirect.github.com/checkstyle/checkstyle/issues/15027">#15027</a> - Add Check Support for Java 21 Pattern Matching for Switch Syntax: WhitespaceAround
<a href="https://redirect.github.com/checkstyle/checkstyle/issues/15025">#15025</a> - Add Check Support for Java 21 Pattern Matching for Switch Syntax: WhitespaceAfter
<a href="https://redirect.github.com/checkstyle/checkstyle/issues/14948">#14948</a> - Add Check Support for Java 21 Record Pattern : ParenPad
<a href="https://redirect.github.com/checkstyle/checkstyle/issues/12542">#12542</a> - new Treewalker property to skip all nested Modules if there is parsing exception
<a href="https://redirect.github.com/checkstyle/checkstyle/issues/14950">#14950</a> - Add Check Support for Java 21 Unnamed Variables <code>&amp;</code> Patterns Syntax: UnusedLocalVariable
<a href="https://redirect.github.com/checkstyle/checkstyle/issues/14963">#14963</a> - Add Check Support for Java 21 Unnamed Variables <code>&amp;</code> Patterns Syntax: FinalParameter
<a href="https://redirect.github.com/checkstyle/checkstyle/issues/14966">#14966</a> - Add Check Support for Java 21 Record Pattern : MethodParamPad
<a href="https://redirect.github.com/checkstyle/checkstyle/issues/14970">#14970</a> - Add Check Support for Java 21 Unnamed Variables <code>&amp;</code> Patterns Syntax: PatternVariableName
<a href="https://redirect.github.com/checkstyle/checkstyle/issues/15015">#15015</a> - Add Check Support for Java 21 Unnamed Variables <code>&amp;</code> Patterns Syntax: CatchParameterName
<a href="https://redirect.github.com/checkstyle/checkstyle/issues/15014">#15014</a> - Add Check Support for Java 21 Unnamed Variables <code>&amp;</code> Patterns Syntax: LambdaParameterName
<a href="https://redirect.github.com/checkstyle/checkstyle/issues/15016">#15016</a> - Add Check Support for Java 21 Unnamed Variables <code>&amp;</code> Patterns Syntax: IllegalIdentifierName
<a href="https://redirect.github.com/checkstyle/checkstyle/issues/15012">#15012</a> - LocalFinalVariableName must include <code>_</code>
<a href="https://redirect.github.com/checkstyle/checkstyle/issues/15053">#15053</a> - Define severity for google style config by variable
<a href="https://redirect.github.com/checkstyle/checkstyle/issues/14946">#14946</a> - Add Check Support for Java 21 Record Pattern : IllegalType
<a href="https://redirect.github.com/checkstyle/checkstyle/issues/14981">#14981</a> - False positive in FinalLocalVariable check on unnamed variables
<a href="https://redirect.github.com/checkstyle/checkstyle/issues/14907">#14907</a> - Add support to check grouping of Constructors in google style guide implementation
<a href="https://redirect.github.com/checkstyle/checkstyle/issues/15059">#15059</a> - Add Check Support for Java 21 Unnamed Variables <code>&amp;</code> Patterns Syntax: New Check UnusedLambdaParameterShouldBeUnnamed
<a href="https://redirect.github.com/checkstyle/checkstyle/issues/15058">#15058</a> - Add Check Support for Java 21 Unnamed Variables <code>&amp;</code> Patterns Syntax: New Check UnusedCatchParameterShouldBeUnnamed
<a href="https://redirect.github.com/checkstyle/checkstyle/issues/14975">#14975</a> - Add Check Support for Java 17 Sealed Classes: New Check SealedShouldHavePermitsList
<a href="https://redirect.github.com/checkstyle/checkstyle/issues/14985">#14985</a> - Add Check Support for Java 21 Pattern Matching for Switch Syntax: New Check WhenShouldBeUsed
<a href="https://redirect.github.com/checkstyle/checkstyle/issues/14979">#14979</a> - Add Check Support for Java 21 Pattern Matching for Switch Syntax: New Check MissingNullCaseInSwitch
<a href="https://redirect.github.com/checkstyle/checkstyle/issues/6723">#6723</a> - new check: LeadingAsteriskAlignCheck</p>
<p>Bug fixes:</p>
<p><a href="https://redirect.github.com/checkstyle/checkstyle/issues/15504">#15504</a> - MissingSwitchDefault false positive on switch expressions when wrapped in return()
<a href="https://redirect.github.com/checkstyle/checkstyle/issues/15445">#15445</a> - Update NPathComplexity Check to Account for when expression as a Possible Execution Path
<a href="https://redirect.github.com/checkstyle/checkstyle/issues/15429">#15429</a> - Google style: method names should not allow to have '_' unless they are annotated by Test
<a href="https://redirect.github.com/checkstyle/checkstyle/issues/15040">#15040</a> - Add Check Support for Java 21 Pattern Matching for Switch Syntax: SimplifyBooleanReturn
<a href="https://redirect.github.com/checkstyle/checkstyle/issues/15322">#15322</a> - NeedBraces Check Should violate Switch Expressions with Arrow Syntax That Directly Yields a Value
<a href="https://redirect.github.com/checkstyle/checkstyle/issues/15047">#15047</a> - Add Check Support for Java 21 Pattern Matching for Switch Syntax: NPathComplexity
<a href="https://redirect.github.com/checkstyle/checkstyle/issues/15045">#15045</a> - Add Check Support for Java 21 Pattern Matching for Switch Syntax:  CyclomaticComplexity
<a href="https://redirect.github.com/checkstyle/checkstyle/issues/15029">#15029</a> - Update EmptyBlockCheck to Violate Empty Case Blocks with Arrow Syntax
<a href="https://redirect.github.com/checkstyle/checkstyle/issues/15123">#15123</a> - MissingSwitchDefault : False positive when <code>case null</code> in switch labeled statement group
<a href="https://redirect.github.com/checkstyle/checkstyle/issues/14688">#14688</a> - LocalVariableNameCheck incorrectly flags variables named <code>_</code> (see JEP456)
<a href="https://redirect.github.com/checkstyle/checkstyle/issues/14891">#14891</a> - Add Support for Java 21 Record Pattern : MissingSwitchDefault
<a href="https://redirect.github.com/checkstyle/checkstyle/issues/14955">#14955</a> - Remove href from LineLength module property in google style config and update test cases</p>
</blockquote>
<p>... (truncated)</p>
</details>
<details>
<summary>Commits</summary>
<ul>
<li><a href="https://github.com/checkstyle/checkstyle/commit/ece521c7c20874165d12e783dcf4fc0fbca1c21b"><code>ece521c</code></a> [maven-release-plugin] prepare release checkstyle-10.18.0</li>
<li><a href="https://github.com/checkstyle/checkstyle/commit/aadb50cb0457c9b8faf3c357be20def96d66b575"><code>aadb50c</code></a> doc: release notes for 10.18.0</li>
<li><a href="https://github.com/checkstyle/checkstyle/commit/3f1fc43c94fa0ae3fca41aa1c04456ead4bb71ec"><code>3f1fc43</code></a> doc: update documentation for jdkVersion property in RedundantModifierCheck</li>
<li><a href="https://github.com/checkstyle/checkstyle/commit/8da6f47dc9f45807931e10e454d6e5191127ba52"><code>8da6f47</code></a> Issue <a href="https://redirect.github.com/checkstyle/checkstyle/issues/15340">#15340</a>: created InputFormatted file for section 3.3.2 No Line Wrapping</li>
<li><a href="https://github.com/checkstyle/checkstyle/commit/7a5837fb06a3228f72b6f60b5bb09e1a6b5d5c55"><code>7a5837f</code></a> Issue <a href="https://redirect.github.com/checkstyle/checkstyle/issues/15340">#15340</a>: created InputFormatted file for section 2.3.1 whitespace charac...</li>
<li><a href="https://github.com/checkstyle/checkstyle/commit/5b2b3305b164ed3ba966952300b88c2d8bcaec4e"><code>5b2b330</code></a> Issue <a href="https://redirect.github.com/checkstyle/checkstyle/issues/15340">#15340</a>: created InputFormatted file for section 7.1.2 Paragraphs</li>
<li><a href="https://github.com/checkstyle/checkstyle/commit/d010bda6ac30f93fa73ec6e4495442fdaaf98c16"><code>d010bda</code></a> Issue <a href="https://redirect.github.com/checkstyle/checkstyle/issues/15159">#15159</a>: RedundantModifierCheck should violate final modifier on unnamed...</li>
<li><a href="https://github.com/checkstyle/checkstyle/commit/d32c27b87650da899104201a82b5220ed6c4234a"><code>d32c27b</code></a> dependency: bump nl.jqno.equalsverifier:equalsverifier</li>
<li><a href="https://github.com/checkstyle/checkstyle/commit/81b0261c31a463dadbc5e99f882c288c687a27f6"><code>81b0261</code></a> Issue <a href="https://redirect.github.com/checkstyle/checkstyle/issues/15059">#15059</a>: New Check UnusedLambdaParameterShouldBeUnnamed</li>
<li><a href="https://github.com/checkstyle/checkstyle/commit/c976754bc12f2c2a391124786ef4d79507055d97"><code>c976754</code></a> Issue <a href="https://redirect.github.com/checkstyle/checkstyle/issues/15340">#15340</a>: created InputFormatted file for section 3 Source File</li>
<li>Additional commits viewable in <a href="https://github.com/checkstyle/checkstyle/compare/checkstyle-10.17.0...checkstyle-10.18.0">compare view</a></li>
</ul>
</details>
<br />

<details>
<summary>Most Recent Ignore Conditions Applied to This Pull Request</summary>

| Dependency Name | Ignore Conditions |
| --- | --- |
| com.puppycrawl.tools:checkstyle | [< 10.13, > 10.12.0] |
</details>

[![Dependabot compatibility score](https://dependabot-badges.githubapp.com/badges/compatibility_score?dependency-name=com.puppycrawl.tools:checkstyle&package-manager=maven&previous-version=10.17.0&new-version=10.18.0)](https://docs.github.com/en/github/managing-security-vulnerabilities/about-dependabot-security-updates#about-compatibility-scores)

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-automerge-start)
[//]: # (dependabot-automerge-end)

---

<details>
<summary>Dependabot commands and options</summary>
<br />

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)

</details>

Closes #2012 from dependabot[bot]/dependabot/maven/java/com.puppycrawl.tools-checkstyle-10.18.0.

Authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Signed-off-by: Dongjoon Hyun <dongjoon@apache.org>
  • Loading branch information
dependabot[bot] authored and dongjoon-hyun committed Aug 26, 2024
1 parent abab96d commit 9097c11
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion java/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -62,7 +62,7 @@
<properties>
<bouncycastle.version>1.78.1</bouncycastle.version>
<brotli4j.version>1.16.0</brotli4j.version>
<checkstyle.version>10.17.0</checkstyle.version>
<checkstyle.version>10.18.0</checkstyle.version>
<example.dir>${project.basedir}/../../examples</example.dir>
<hadoop.version>3.4.0</hadoop.version>
<java.version>17</java.version>
Expand Down

0 comments on commit 9097c11

Please sign in to comment.