Skip to content

chore(deps): Update dependency com.puppycrawl.tools:checkstyle to v12#617

Merged
zml2008 merged 1 commit intoSpongePowered:trunkfrom
stoyanK7:major-checkstyle-12
Jan 26, 2026
Merged

chore(deps): Update dependency com.puppycrawl.tools:checkstyle to v12#617
zml2008 merged 1 commit intoSpongePowered:trunkfrom
stoyanK7:major-checkstyle-12

Conversation

@stoyanK7
Copy link
Contributor

Updated Checkstyle version from 11.1.0 to latest 12.3.0.

Checkstyle 12.3.0 release notes

This pull request is same as PR #613, but also fixes the 7 Checkstyle violations from that PR:

Error: eckstyle] [ERROR] /home/runner/work/Configurate/Configurate/core/src/main/java/org/spongepowered/configurate/loader/package-info.java:17: First sentence of Javadoc is missing an ending period. [SummaryJavadoc]
Error: eckstyle] [ERROR] /home/runner/work/Configurate/Configurate/core/src/main/java/org/spongepowered/configurate/reactive/CachedPublisher.java:26: First sentence of Javadoc is missing an ending period. [SummaryJavadoc]
Error: eckstyle] [ERROR] /home/runner/work/Configurate/Configurate/core/src/main/java/org/spongepowered/configurate/serialize/Scalars.java:23: First sentence of Javadoc is missing an ending period. [SummaryJavadoc]
Error: eckstyle] [ERROR] /home/runner/work/Configurate/Configurate/core/src/main/java/org/spongepowered/configurate/serialize/NumericSerializers.java:22: First sentence of Javadoc is missing an ending period. [SummaryJavadoc]
Error: eckstyle] [ERROR] /home/runner/work/Configurate/Configurate/core/src/main/java/org/spongepowered/configurate/package-info.java:17: First sentence of Javadoc is missing an ending period. [SummaryJavadoc]
Error: eckstyle] [ERROR] /home/runner/work/Configurate/Configurate/core/src/main/java/org/spongepowered/configurate/transformation/package-info.java:17: First sentence of Javadoc is missing an ending period. [SummaryJavadoc]
Error: eckstyle] [ERROR] /home/runner/work/Configurate/Configurate/core/src/main/java/org/spongepowered/configurate/transformation/TransformAction.java:49: First sentence of Javadoc is missing an ending period. [SummaryJavadoc]

Related to checkstyle/checkstyle#18301 where we want to bring Configurate project back into Checkstyle regression suite. See checkstyle/checkstyle#18301 (comment)

@romani
Copy link

romani commented Dec 17, 2025

@zml2008 , I am supporting this update.
We merged restoration in CI of your project.

@zml2008
Copy link
Member

zml2008 commented Jan 26, 2026

Thank you!

@zml2008 zml2008 self-assigned this Jan 26, 2026
@zml2008 zml2008 added the tooling Configurate's own build process, documentation, and other code items not part of a specific module. label Jan 26, 2026
@zml2008 zml2008 added this to the 4.3.0 milestone Jan 26, 2026
@zml2008 zml2008 merged commit c09f48a into SpongePowered:trunk Jan 26, 2026
This was referenced Feb 28, 2026
renovate-sh-app bot added a commit to grafana/grafana-opentelemetry-java that referenced this pull request Mar 3, 2026
This PR contains the following updates:

| Package | Change |
[Age](https://docs.renovatebot.com/merge-confidence/) |
[Confidence](https://docs.renovatebot.com/merge-confidence/) |
|---|---|---|---|
| [checkstyle](https://checkstyle.org/)
([source](https://redirect.github.com/checkstyle/checkstyle)) | `13.2.0`
→ `13.3.0` |
![age](https://developer.mend.io/api/mc/badges/age/maven/com.puppycrawl.tools:checkstyle/13.3.0?slim=true)
|
![confidence](https://developer.mend.io/api/mc/badges/confidence/maven/com.puppycrawl.tools:checkstyle/13.2.0/13.3.0?slim=true)
|

---

> [!WARNING]
> Some dependencies could not be looked up. Check the [Dependency
Dashboard](../issues/1133) for more information.

---

### Release Notes

<details>
<summary>checkstyle/checkstyle (checkstyle)</summary>

###
[`v13.3.0`](https://redirect.github.com/checkstyle/checkstyle/releases/tag/checkstyle-13.3.0)

[Compare
Source](https://redirect.github.com/checkstyle/checkstyle/compare/checkstyle-13.2.0...checkstyle-13.3.0)

Checkstyle 13.3.0 -
<https://checkstyle.org/releasenotes.html#Release_13.3.0>

Breaking backward compatibility:


[#&#8203;18146](https://redirect.github.com/checkstyle/checkstyle/issues/18146)
- Clean up grammar for discontinued preview feature (pattern matching
for switch)

New:


[#&#8203;15180](https://redirect.github.com/checkstyle/checkstyle/issues/15180)
- NewCheck: UseEnhancedSwitch

[#&#8203;18494](https://redirect.github.com/checkstyle/checkstyle/issues/18494)
- New Check: GoogleNonConstantFieldNameCheck to enforce Google Java
Style Guide member naming

[#&#8203;18109](https://redirect.github.com/checkstyle/checkstyle/issues/18109)
- new Check: LineEnding

Bug fixes:


[#&#8203;9719](https://redirect.github.com/checkstyle/checkstyle/issues/9719)
- Indentation check regression with "new"

[#&#8203;17541](https://redirect.github.com/checkstyle/checkstyle/issues/17541)
- Empty line separator is not enforced in `package-info.java` file

<details>
<summary>Other Changes:</summary>
<br/>
  Enforce file size on Java inputs <br/>
Re-enable inspections that were temporarily suppressed for Qodana <br/>
  Fix  PMD 7.1.0 Violations <br/>
  Add missing property examples to XDocs for Checkstyle Checks <br/>
  Pitest: examine and try to remove infinite loop mutations <br/>
Remove from whitelist file odd words that came from test imput files
<br/>
Fix xdocs Examples AST Consistency Test (Reduce suppressions list) <br/>
Resolve violation from RegExpDuplicateCharacterInClass inspection <br/>
SarifLoggerTest.java to use verifyWithInlineConfigParserAndLogger <br/>
  Resolve Pitest Suppressions - java-ast-visitor <br/>
Conver test from XpathFilterElementTest to
SuppressionXpathSingleFilterTest to use inlined config in Input files
<br/>
Update `JavadocCommentsTokenTypes.java` to new format of AST print <br/>
  Expand "Writing Check" to include XPath support <br/>
  Top comment should 'Compilable with Java22' <br/>
False positive: indentation inside of constructor parameters inside try
block <br/>
Updating properties in Input files to mention all default properties,
<br/>
  Resolve error-prone violations <br/>
Organize openrewrite staticanalysis composite recipes by groups as it is
done on openrewrite website <br/>
  Define violation messages for all violations <br/>
  Find way to keep qodana yml in config folder <br/>
  Fix typos and improve grammer in testing techniques guide <br/>
doc: Add DESCRIPTION token example with AST tree - Issue #&#8203;17882
<br/>
Clone `SpongePowered/Configurate` once SpongePowered/Configurate#617 is
merged <br/>
Documentation examples for RegexpHeader and Header does not have any
header <br/>
  Activate Java 21 migration recipes in OpenRewrite CI <br/>
  Consolidate `no_output_timeout` <br/>
  Resolve CheckerFramework violations <br/>
  migrate IDEA inpection in CI to most recent version <br/>
  Fix comment <br/>
</details>

</details>

---

### Configuration

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

🚦 **Automerge**: Enabled.

♻ **Rebasing**: Whenever PR is behind base branch, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

## Need help?
You can ask for more help in the following Slack channel:
#proj-renovate-self-hosted. In that channel you can also find ADR and
FAQ docs in the Resources section.

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiI0My4zMC4xIiwidXBkYXRlZEluVmVyIjoiNDMuMzAuMSIsInRhcmdldEJyYW5jaCI6Im1haW4iLCJsYWJlbHMiOlsidXBkYXRlLW1pbm9yIl19-->

Signed-off-by: renovate-sh-app[bot] <219655108+renovate-sh-app[bot]@users.noreply.github.com>
Co-authored-by: renovate-sh-app[bot] <219655108+renovate-sh-app[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

tooling Configurate's own build process, documentation, and other code items not part of a specific module.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants