[To be discussed] Migration to new checkstyle#3736

Draft

rolnico wants to merge 20 commits intomainfrom

nro/new_checkstyle

Member

rolnico commented Jan 26, 2026

Please check if the PR fulfills these requirements

The commit message follows our guidelines
Tests for the changes have been added (for bug fixes / features)
Docs have been added / updated (for bug fixes / features)
A PR or issue has been opened in all impacted repositories (if any)

Does this PR already have an issue describing the problem?
Linked to powsybl/powsybl-parent#90

What kind of change does this PR introduce?

What is the current behavior?

What is the new behavior (if this is a feature change)?

Does this PR introduce a breaking change or deprecate an API?

Yes
No

If yes, please check if the following requirements are fulfilled

The Breaking Change or Deprecated label has been added
The migration steps are described in the following section

What changes might users need to make in their application due to this PR? (migration steps)

Other information:

rolnico added 18 commits

December 8, 2025 08:56


          Fix new issues after checkstyle changes, up to iidm-test

df299c7

Signed-off-by: Nicolas Rol <nicolas.rol@rte-france.com>


          Continue fixing the issues

7af6fa2

Signed-off-by: Nicolas Rol <nicolas.rol@rte-france.com>


          Fix new issues after checkstyle changes, up to iidm-impl and fix some…

e88184e

… classes

Signed-off-by: Nicolas Rol <nicolas.rol@rte-france.com>


          Fix FourSubstationsNodeBreakerFactory

c4a8363

Signed-off-by: Nicolas Rol <nicolas.rol@rte-france.com>


          Fix new issues after checkstyle changes, up to iidm-serde

b4bcfe7

Signed-off-by: Nicolas Rol <nicolas.rol@rte-france.com>


          Fix new issues after checkstyle changes, up to loadflow-api

e1a6656

Signed-off-by: Nicolas Rol <nicolas.rol@rte-france.com>


          Fix new issues after checkstyle changes, up to iidm-modification

a5156e5

Signed-off-by: Nicolas Rol <nicolas.rol@rte-france.com>


          Fix new issues after checkstyle changes, up to security-analysis

430349c

Signed-off-by: Nicolas Rol <nicolas.rol@rte-france.com>


          Fix new issues after checkstyle changes, up to security-analysis-api

8cbed7d

Signed-off-by: Nicolas Rol <nicolas.rol@rte-france.com>


          Fix new issues after checkstyle changes, up to triple-store

5a12130

Signed-off-by: Nicolas Rol <nicolas.rol@rte-france.com>


          Fix new issues after checkstyle changes, up to loadflow-results-compl…

b861364

…etion

Signed-off-by: Nicolas Rol <nicolas.rol@rte-france.com>


          Fix new issues after checkstyle changes, up to cgmes-completion

28e6fc3

Signed-off-by: Nicolas Rol <nicolas.rol@rte-france.com>


          Fix new issues after checkstyle changes, up to cgmes-alternatives

bcfd2be

Signed-off-by: Nicolas Rol <nicolas.rol@rte-france.com>


          Fix new issues after checkstyle changes

f809ef1

Signed-off-by: Nicolas Rol <nicolas.rol@rte-france.com>


          revert parent version to 24

3a2aeba

Signed-off-by: Nicolas Rol <nicolas.rol@rte-france.com>


          Modify @SuppressWarnings("checkstyle:IllegalCatch") to IllegalCatchEr…

efa70c0

…ror and IllegalCatchWarning

Signed-off-by: Nicolas Rol <nicolas.rol@rte-france.com>


          Merge branch 'main' into nro/new_checkstyle

# Conflicts:
#	commons/src/main/resources/com/powsybl/commons/reports_fr.properties
#	computation-local/src/test/java/com/powsybl/computation/local/ProcessHelperTest.java
#	iidm/iidm-impl/src/main/java/com/powsybl/iidm/network/impl/VoltageLevelImpl.java


          fix reports_fr.properties due to migration to UTF-8 encoding

d37ac42

Signed-off-by: Nicolas Rol <nicolas.rol@rte-france.com>

rolnico self-assigned this

rolnico added the Quality label

rolnico added 2 commits

March 11, 2026 16:50


          Merge branch 'main' into nro/new_checkstyle

1b25b14

# Conflicts:
#	action-api/src/main/java/com/powsybl/action/json/ActionJsonModule.java
#	action-api/src/test/java/com/powsybl/action/json/JsonActionTest.java
#	cgmes/cgmes-conformity/src/main/java/com/powsybl/cgmes/conformity/CgmesConformity1NetworkCatalog.java
#	cgmes/cgmes-conversion/src/main/java/com/powsybl/cgmes/conversion/Conversion.java
#	cgmes/cgmes-conversion/src/main/java/com/powsybl/cgmes/conversion/elements/EquivalentInjectionConversion.java
#	cgmes/cgmes-conversion/src/main/java/com/powsybl/cgmes/conversion/elements/transformers/TwoWindingsTransformerConversion.java
#	cgmes/cgmes-conversion/src/main/java/com/powsybl/cgmes/conversion/export/EquipmentExport.java
#	cgmes/cgmes-conversion/src/main/java/com/powsybl/cgmes/conversion/export/TopologyExport.java
#	cgmes/cgmes-conversion/src/main/java/com/powsybl/cgmes/conversion/naming/AbstractCgmesAliasNamingStrategy.java
#	cgmes/cgmes-conversion/src/main/java/com/powsybl/cgmes/conversion/naming/SimpleCgmesAliasNamingStrategy.java
#	cgmes/cgmes-conversion/src/test/java/com/powsybl/cgmes/conversion/test/export/ExportXmlCompare.java
#	cgmes/cgmes-extensions/src/main/java/com/powsybl/cgmes/extensions/compatibility/CgmesControlAreasSerDe.java
#	cgmes/cgmes-extensions/src/test/java/com/powsybl/cgmes/extensions/compatibility/CgmesControlAreasSerDeTest.java
#	cgmes/cgmes-gl/src/main/java/com/powsybl/cgmes/gl/LinePositionImporter.java
#	commons/src/main/resources/com/powsybl/commons/reports.properties
#	commons/src/main/resources/com/powsybl/commons/reports_fr.properties
#	contingency/contingency-api/src/test/java/com/powsybl/contingency/BoundaryLineContingencyTest.java
#	contingency/contingency-api/src/test/java/com/powsybl/contingency/tasks/BoundaryLineTrippingTest.java
#	iidm/iidm-api/src/main/java/com/powsybl/iidm/network/Bus.java
#	iidm/iidm-api/src/main/java/com/powsybl/iidm/network/TieLine.java
#	iidm/iidm-api/src/main/java/com/powsybl/iidm/network/ValidationUtil.java
#	iidm/iidm-api/src/main/java/com/powsybl/iidm/network/VoltageLevel.java
#	iidm/iidm-api/src/main/java/com/powsybl/iidm/network/util/BoundaryLineData.java
#	iidm/iidm-api/src/main/java/com/powsybl/iidm/network/util/TieLineUtil.java
#	iidm/iidm-api/src/test/java/com/powsybl/iidm/network/util/BoundaryLineDataTest.java
#	iidm/iidm-api/src/test/java/com/powsybl/iidm/network/util/BoundaryLineTestData.java
#	iidm/iidm-criteria/src/test/java/com/powsybl/iidm/criteria/DanglingLineCriterionTest.java
#	iidm/iidm-impl/src/test/java/com/powsybl/iidm/network/impl/util/SVTest.java
#	iidm/iidm-modification/src/test/java/com/powsybl/iidm/modification/scalable/BoundaryLineScalableTest.java
#	iidm/iidm-modification/src/test/java/com/powsybl/iidm/modification/scalable/ProportionalScalableTest.java
#	iidm/iidm-serde/src/main/java/com/powsybl/iidm/serde/AbstractTreeDataExporter.java
#	iidm/iidm-serde/src/main/java/com/powsybl/iidm/serde/DanglingLineSerDe.java
#	iidm/iidm-serde/src/main/java/com/powsybl/iidm/serde/ShuntSerDe.java
#	iidm/iidm-serde/src/main/java/com/powsybl/iidm/serde/TieLineSerDe.java
#	iidm/iidm-tck/src/test/java/com/powsybl/iidm/network/tck/AbstractBoundaryLineTest.java
#	psse/psse-converter/src/test/java/com/powsybl/psse/converter/PsseFullExportTest.java
#	security-analysis/security-analysis-api/src/main/java/com/powsybl/security/json/limitreduction/LimitReductionDeserializer.java


          Fix merge

ba1696d

Signed-off-by: Nicolas Rol <nicolas.rol@rte-france.com>

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

Labels