-
Notifications
You must be signed in to change notification settings - Fork 21
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Implementing Checkstyle #208
Conversation
Is there any new progress? I suggest moving this PR to 1.1.3. |
Got it. Let me move this and others. |
@mirromutth |
8f673b3
to
bbbc32c
Compare
bbbc32c
to
2dd6512
Compare
2dd6512
to
f90291a
Compare
524e049
to
6bcfa9f
Compare
6bcfa9f
to
9421407
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Motivation: To ensure code consistency and quality through the adoption of Checkstyle Modification: Added Checkstyle plugin and configuration, updated code to comply with new coding standards. Set version as INTERNAL on internal modules Result: Enhanced code quality and consistency across the project, facilitating easier contributions and maintenance.
9421407
to
9e996ad
Compare
Motivation:
To ensure code consistency and quality through the adoption of Checkstyle
Modification:
Added Checkstyle plugin and configuration, updated code to comply with new coding standards.
Set version as INTERNAL on internal modules
Result:
Enhanced code quality and consistency across the project, facilitating easier contributions and maintenance.