-
Notifications
You must be signed in to change notification settings - Fork 26
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
Add java format checker to github Actions #262
Conversation
<title>lpvs</title><script type="text/javascript" src="jacoco-resources/sort.js"></script> Sessionslpvs lpvs
Created with JaCoCo 0.8.8.202204050719
|
3f4d69f
to
af7ddc1
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.
What about using AOSP style (4-space indentation)?
<title>lpvs</title><script type="text/javascript" src="jacoco-resources/sort.js"></script> Sessionslpvs lpvs
Created with JaCoCo 0.8.8.202204050719
|
af7ddc1
to
16dc3e3
Compare
<title>lpvs</title><script type="text/javascript" src="jacoco-resources/sort.js"></script> Sessionslpvs lpvs
Created with JaCoCo 0.8.8.202204050719
|
114f0f5
to
ee2e1f6
Compare
<title>lpvs</title><script type="text/javascript" src="jacoco-resources/sort.js"></script> Sessionslpvs lpvs
Created with JaCoCo 0.8.8.202204050719
|
7ab3067
to
d03c3d1
Compare
<title>lpvs</title><script type="text/javascript" src="jacoco-resources/sort.js"></script> Sessionslpvs lpvs
Created with JaCoCo 0.8.8.202204050719
|
700aded
to
d8869a4
Compare
<title>lpvs</title><script type="text/javascript" src="jacoco-resources/sort.js"></script> Sessionslpvs lpvs
Created with JaCoCo 0.8.8.202204050719
|
Signed-off-by: Taras Drozdovskyi <[email protected]>
e1f15a2
to
f114322
Compare
<title>lpvs</title><script type="text/javascript" src="jacoco-resources/sort.js"></script> Sessionslpvs lpvs
Created with JaCoCo 0.8.8.202204050719
|
Done! |
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
Code style changes will be applied after merging all opened PRs. |
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
Description
You need to add a checker to match the style code
Type of change
How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration. Below is just an example and you can delete it when you contribute.
See github->actions
Checklist: