Terasology/facades
jdrueckert ba4d716488
qa: address pmd findings (#5274)
* qa: fix PMD CollapsibleIfStatements finding

* qa: fix PMD SimplifiedTernary findings

* fix: formatting mistake

* qa: address PMD SystemPrintln findings

- suppress for LoggingContext.java and Terasology.java because logging not necessarily available yet there
- refactor for PathManager.java as logger is already in use in that context, also see previous discussion in https://github.com/MovingBlocks/Terasology/pull/5036#discussion_r889518509

* qa: address PMD InvalidLogMessageFormat finding

- if I understand https://stackoverflow.com/questions/7102339/is-there-a-correct-way-to-pass-arguments-in-slf4j correctly, this should not be less performant but valid SLF4j format

* qa: address PMD AvoidBranchingStatementAsLastInLoop findings

* qa: address PMD ForLoopCanBeForeach findings

* chore: remove unused import

* qa: address PMD AvoidPrintStackTrace findings

* chore: remove unused imports

* chore: fix checkstyle DeclarationOrderCheck warning
2024-10-04 08:11:42 +02:00
..
PC qa: address pmd findings (#5274) 2024-10-04 08:11:42 +02:00
subprojects.settings.gradle.kts build: migrate subprojects.settings.gradle to kotlin (#5203) 2024-01-08 19:16:47 +00:00