Roman Ivanov

icon indicating a website link http://roman-ivanov.blogspot.com

Results 1611 comments of Roman Ivanov

Issue #17299: Convert eligible classes to records

compilation problems

Issue #17299: Convert eligible classes to records

@AmitKumarDeoghoria , please rebase, resolve conflict and remove update for API classes.

Issue #17299: Convert eligible classes to records

Compilation errors, please fix

Rule for enforcing number of blank lines near class declarations?

@mcandre , please give us more details on what is expected. Just imagine that you already have a check how you would like to customize it and what code should...

minimum supported JDK version to run Error Prone is JDK 21 as of Error Prone 2.43.0

We already on jdk21 in CI

minimum supported JDK version to run Error Prone is JDK 21 as of Error Prone 2.43.0

We are not yet ready for jdk21 syntax. We can start using new syntax after https://github.com/checkstyle/checkstyle/pull/17669 is merged , January 2026 .

Issue #17554: Report UnneccessaryParentheses for dot, 'new' and array access

To avoid CI failure https://checkstyle.semaphoreci.com/jobs/7843909d-eb87-4fc1-bef2-7669b076d6b2#L639 Please send PR to https://github.com/checkstyle/contribution/blob/master/patch-diff-report-tool/src/test/java/com/github/checkstyle/internal/AbstractTest.java

Issue #17554: Report UnneccessaryParentheses for dot, 'new' and array access

Github, generate report for UnnecessaryParentheses/all-examples-in-one

Issue #17554: Report UnneccessaryParentheses for dot, 'new' and array access

@zbynek , please do spot checking of diff report.

Issue #17554: Report UnneccessaryParentheses for dot, 'new' and array access

diff between reports. https://www.diffchecker.com/FMphnMXV/ please add to our Inputs cases that forced you to remove token LITERAL_NEW . We try to not rely much on diff regression and have all...