Praveen Kumar comments

Results 29 comments of


                                            Praveen Kumar

[Bug] Checkstyle check is not applied to all tests

If no one is currently working on this issue, I'm happy to work on it. As expected, the errors can be seen below: ``` $ ./mvnw initialize checkstyle:check -pl pulsar-broker...

[Bug] Checkstyle check is not applied to all tests

@BewareMyPower , Do we need to identify all checks or only specific ones? The error above shows only an ImportOrder violation.

[Bug] Checkstyle check is not applied to all tests

> It should be all checks, which should follow `buildtools/src/main/resources/pulsar/checkstyle.xml` ok, i guess i taking time to fix all the errors.

[Bug] Checkstyle check is not applied to all tests

sorry, i have my final year exams, so i was not giving enough time to it, after exam i will request a pull request

[Bug] Checkstyle check is not applied to all tests

@Bhargavkonidena i m already working on it.

[Bug] Checkstyle check is not applied to all tests

Most of the Checkstyle violations are related to ImportOrder, No spaces between import statements, LineLength, whitespace issues, and some naming conventions for final fields and class names. After fixing these...

[Bug] Checkstyle check is not applied to all tests

@lhotari please review my PR

Issue #17778: Added JavadocMissingLeadingAsterisk and JavadocContentLocation Checks in google_checks.xml for Rule: 7.1.1 General Form

GitHub, generate website

Issue #17778: Added JavadocMissingLeadingAsterisk and JavadocContentLocation Checks in google_checks.xml for Rule: 7.1.1 General Form

@romani please, review this PR

Issue #17778: Added JavadocMissingLeadingAsterisk and JavadocContentLocation Checks in google_checks.xml for Rule: 7.1.1 General Form

@Zopsss Please take a look at issues #17695 and #17832. I think this PR also helps to close them. I have created the remaining cases from the 7.1.1 general form...