Roman Ivanov
Roman Ivanov
@Kevin222004 , CI is red
Javadoc are very special group, we less demanding to prove that code is required. We know already that implementation of such checks are very error prone, so if regression can...
please read PR description and stricktly follow it.
Better for us to stay on jdk 11 version and migrate to jdk 17 after summer . According to last migration to jdk11, we agreed on rule of migration: after...
from Sonar: https://docs.sonarsource.com/sonarqube/latest/requirements/prerequisites-and-overview/ > For the SonarScanners, the minimum recommended version is Java 17. Java 11 is deprecated as scanner runtime environment. so we are absolute good to migrate to...
From sonar web site https://sonarcloud.io/project/issues?resolved=false&types=CODE_SMELL&sinceLeakPeriod=true&id=org.checkstyle%3Acheckstyle >This project was scanned using Java 11 runtime. Starting from Jan 15, 2024, SonarCloud will [stop accepting scans](https://community.sonarsource.com/t/java-11-is-deprecated-as-a-runtime-env-to-scan-your-projects/96597) that are started with Java 11. We...
please investigate `finishLocalSetup` we can move validation of ctor parameters to this method, we just need some initialiation method that is called before all.
>OutputStreamWriter not receives a null stream, it will throw an NPE. I thought we did all to prove that npe is not possible, we have Junit pure unit tests. @relentless-pursuit...
@relentless-pursuit , please add Junit for `final DefaultLogger dl = new DefaultLogger(null, null, null, null);` It is a a case that was problem even before, but @rnveach raising it as...