Roman Ivanov comments

Results 1611 comments of


                                            Roman Ivanov

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

I understand general benefits of new methods, It is first time we run into this, and when I did suggested code it looked weird, we over care to readability of...

semgrep ci [targets] is not supported

@nrmancuso , what jdk syntax we support now ? 19th ? do we have full support or partial ?

doc: BEGINNING_DEVELOPMENT.md

There are more room to improve this doc we will do in separate PR.

Issue #14937: Migrated section 7.1.1 General Form to chapter wise testing

@Zopsss , CI is red, please investigate `[ERROR] InvalidJavadocPositionTest.testDefault:65->AbstractItModuleTestSupport.getLinesWithWarn:618 » NoSuchFile /home/vsts/work/1/s/src/it/resources/com/google/checkstyle/test/chapter7javadoc/rule711generalform/InputInvalidJavadocPosition.java`

Document System Variables usage for Windows OS

Let's extend documentation, this is crazy nuance, no much people know about it.

Document System Variables usage for Windows OS

For me this nuance is very non intuitive. @nrmancuso , what is your opinion on this?

Document System Variables usage for Windows OS

@nrmancuso , PR is already opened, please look.

New check ChainedMethodCallWrap

Last comment was about to move it to https://github.com/sevntu-checkstyle/sevntu.checkstyle , we just need to find someone to copy code to other repo.

New check ChainedMethodCallWrap

We are waiting for help to move referenced implementation to https://github.com/sevntu-checkstyle/sevntu.checkstyle , to let us release it and get feedback on usage of it, as there are nuances.

Issue #5138: rename SINGLETON_ELEMENT to VOID_HTML_ELEMENT for clarity

we migrated to new TokenTypes