-
Notifications
You must be signed in to change notification settings - Fork 1
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
Added Check Style #1546
Added Check Style #1546
Conversation
90d441f
to
cb9d08b
Compare
fb83bc8
to
bf43c97
Compare
...ration_api/jobs/executors/associations/alleleAssociations/AlleleGeneAssociationExecutor.java
Outdated
Show resolved
Hide resolved
@adamgibs this is only for *.java files. I would like to set this up for *.js files but the rules would be different. Maybe we can find an example checkstyle.xml for *.js? |
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.
Could you add to the README a line explaining how to use it locally before pushing?
@oblodgett It looks like checkstyle is specifically for Java. I'm not seeing any examples of using with with JavaScript files. |
|
||
@Inject | ||
PersonDAO personDAO; | ||
@Inject PersonDAO personDAO; |
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.
I like to have the annotation on top of the line it's annotating as it is easier to read for me.
Can we not change this?
@IndexedEmbedded( | ||
includePaths = { | ||
"functionalImpacts.name", "phenotypeTerm.curie", "phenotypeTerm.name", "phenotypeStatement", "evidence.curie", | ||
"functionalImpacts.name_keyword", "phenotypeTerm.curie_keyword", "phenotypeTerm.name_keyword", "phenotypeStatement_keyword", |
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.
Nice!
This PR add check style which checks the formatting of the code. This will alert to extraneous space in the code or spaces vs tabs, etc. There is 3 major changes (although minor in the grand scheme of things)