-
Notifications
You must be signed in to change notification settings - Fork 481
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
ORC-982: Extract checkstyle to a single file, help newcomers check code style #895
Conversation
java/checkstyle.xml
Outdated
"https://checkstyle.org/dtds/configuration_1_2.dtd"> | ||
|
||
<module name="Checker"> | ||
<module name="FileTabCharacter"> |
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.
Shall we use two-space indentation?
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.
My IDE's config for xml indentation defaults to 4 spaces. I'll fix this later.
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.
Fix in ec7c471.
cc @williamhyun |
java/checkstyle.xml
Outdated
limitations under the License. | ||
--> | ||
<!DOCTYPE module PUBLIC | ||
"-//Checkstyle//DTD Checkstyle Configuration 1.2//EN" |
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.
ditto. This needs 4 spaces instead of 8 spaces.
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.
+1, LGTM. Merged to main.
Thank you, @guiyanakuang .
Thank you very much @dongjoon-hyun, thank to review and fix format. |
What changes were proposed in this pull request?
Extract checkstyle to a single file.
Added tips to coding.md.
Why are the changes needed?
CheckStyle-IDEA plugin is very simple to load this checkstyle.xml. This way you get checkstyle errors/warnings already when you are coding.
How was this patch tested?
Pass the CIs.