Skip to content
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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Peagram
Copy link

@Peagram Peagram commented Oct 20, 2022

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

Comment on lines +120 to +121
"Unable to handle request ({}), the active session was not found for"
+ " {}.",
Copy link
Member

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.

Comment on lines +56 to +57
private final Map<
String, AbstractMap.SimpleImmutableEntry<String, CompletableFuture<Confirmation>>>
Copy link
Member

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
Copy link
Member

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?

@TVolden
Copy link
Member

TVolden commented Jan 27, 2023

Sorry I forgot to post my comments.

@Peagram
Copy link
Author

Peagram commented Feb 1, 2023

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants