Roman Ivanov
Roman Ivanov
Please share html report of coverage in your GitHub.io
https://kalpadiptyaroy.github.io/com.puppycrawl.tools.checkstyle.checks.indentation/LineWrappingHandler.java.html#L302  in first case, interpretation always goes through where `parentNode.getPreviousSibling()` is null always or not null always. Find a test Input to make both, if not possible remove this...
No need to waste CI, just run `mvn clean test` on your local
Please read several articles on code coverage to understand what you are fighting with. Please do not push until you get 100% on your local, CI will have same results...
Please fix CI https://dev.azure.com/romanivanovjr/romanivanovjr/_build/results?buildId=19880&view=logs&j=c902ebb4-c9f8-5f09-4e17-ff78fbbc842e&t=9ca98c81-ff64-58f0-9d03-a23ac1c4a111&l=1330 Before push validate on on your local by `mvn clean verify `
Already done by somebody else
CI is red https://dev.azure.com/romanivanovjr/romanivanovjr/_build/results?buildId=19946&view=logs&j=c902ebb4-c9f8-5f09-4e17-ff78fbbc842e&t=9ca98c81-ff64-58f0-9d03-a23ac1c4a111&l=1397 Please make sure `mvn clean verify ` pass on local before push . If you don't know what to do to fix, let us know.
Let's restate issue, from `Add support to ignore JSNI methods for LineLength check` to `Add support to ignore JSNI methods for Google style config` CLI execution should show usage of...
Yes, plus: Update xdoc file for style guide to remove comment that we are not covering it.
>Why do we write some test cases in it folder? to be sure that jav code written in Input file has violation or has no violation, and does not matter...