Skip to content

Conversation

@stoyanK7
Copy link

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants