Skip to content

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
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