Roman Ivanov comments

Results 1611 comments of


                                            Roman Ivanov

Give violation for preceding whitespaces for paragraph tag content

Removal of comments "until" is done. @Anmol202005 , thanks a lot for this.

Resolve `TailRecursion` inspection violations by replacing tail recursive calls

Better to deal with each violation (recursion case) in separate PR. It might be tricky to fix.

Resolve `TailRecursion` inspection violations by replacing tail recursive calls

@mohitsatr , you can do it, but we highly recommend to finish all kinds of issues that labeled "good xxxx issue"

Resolve `TailRecursion` inspection violations by replacing tail recursive calls

@mohitsatr , There is last case https://github.com/checkstyle/checkstyle/blob/f812fceef8db0d65d122f764513b687b05d9ae72/src/main/java/com/puppycrawl/tools/checkstyle/api/FullIdent.java#L78 Do you have time to finish it?

Resolve `TailRecursion` inspection violations by replacing tail recursive calls

Issue is closed , I see no other suppressions https://github.com/search?q=repo%3Acheckstyle%2Fcheckstyle%20TailRecursion&type=code

Resolve `TailRecursion` inspection violations by replacing tail recursive calls

@mohitsatr, thanks a lot for your help!!

Issue #12242: expand AuditEvents for fileContent

@Harsh-Agrawal96 , thanks a lot for update. Please make CI green and what first usage will be for fileContent to be good example of how valuable is this. can we...

Issue #12242: expand AuditEvents for fileContent

do you think we do not need this this any more ?

Issue #12242: expand AuditEvents for fileContent

@Harsh-Agrawal96 , I updated issue description. I still we need it, as it would help us to not to cache of file content, as it is already in memory and...

Is there way to avoid 'Prefer java.util.Optional.orElseThrow' if 'isPresent()' is right before

is there a way to disable specific validator ? I did: ``` java/util/Optional.get:.* ``` based on https://github.com/gaul/modernizer-maven-plugin/blob/a553adf84d6b16b9e18f3d1365afc68cea950fb5/modernizer-maven-plugin/src/main/resources/modernizer.xml#L1293C13-L1293C56