-
Notifications
You must be signed in to change notification settings - Fork 942
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
Making tests happy. #100
Making tests happy. #100
Conversation
PTAL @markmcd |
contexts.add(new Object[]{context}); | ||
} | ||
|
||
if (supportsClientId) { | ||
if (!(System.getenv("CLIENT_ID") == null | ||
|| System.getenv("CLIENT_SECRET") == null)) { |
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.
nit: continuation spacing here should be +4
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.
Do we have a code style import method for IntelliJ?
|| System.getProperty("client.secret") == null || System.getProperty("client.secret").equals(""))) { | ||
GeoApiContext context = new GeoApiContext() | ||
.setEnterpriseCredentials(System.getProperty("client.id"), System.getProperty("client.secret")); | ||
contexts.add(new Object[]{context}); |
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.
There must be some way to make this read more nicely. Maybe some local variables?
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've introduced local vars. What line length limit are we using for this project?
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.
lg. i think 100-120 is reasonable.
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.
Thanks Cbro!
Also trimming whitespace from README.md.