-
Notifications
You must be signed in to change notification settings - Fork 176
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
Fixed GoogleJavaFormat using google-java-format-1.15.0 #219
base: master
Are you sure you want to change the base?
Conversation
"Unable to handle request ({}), the active session was not found for" | ||
+ " {}.", |
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.
Maybe skip the strick line length here, it doesn't really improve readability.
private final Map< | ||
String, AbstractMap.SimpleImmutableEntry<String, CompletableFuture<Confirmation>>> |
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.
Again, the strict line length makes the code less readable.
@@ -59,20 +59,24 @@ public interface ClientSecurityExtEventHandler { | |||
DeleteCertificateConfirmation handleDeleteCertificateRequest(DeleteCertificateRequest request); | |||
|
|||
/** | |||
* Handle a {@link ExtendedTriggerMessageRequest} and return a {@link ExtendedTriggerMessageConfirmation}. | |||
* Handle a {@link ExtendedTriggerMessageRequest} and return a {@link |
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.
Does the link still work?
Sorry I forgot to post my comments. |
Hi @TVolden, I am no longer working on this project. I agree with you that some lines are less readable, but they are like the generator leave (google java format). I did these modification because without do that, gradle didn't work |
When I try to build the project using gradle, I receive an error with the format of some files, I was fixed the format of those files