Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Comments

Close side panel

Formatting and making some methods private#417

Merged
smirnoal merged 1 commit intomainaws/aws-lambda-java-libs:mainfrom
formattingaws/aws-lambda-java-libs:formattingCopy head branch name to clipboard
Apr 7, 2023
Merged

Formatting and making some methods private#417
smirnoal merged 1 commit intomainaws/aws-lambda-java-libs:mainfrom
formattingaws/aws-lambda-java-libs:formattingCopy head branch name to clipboard

Conversation

@smirnoal
Copy link
Contributor

@smirnoal smirnoal commented Apr 4, 2023

Description of changes:

  • Reformatted sources with idea default formatter.
  • Revisited methods visibility in EventHandlerLoader

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Apply formatting, revisit methods visibility in `EventHandlerLoader`

import com.amazonaws.services.lambda.runtime.api.client.util.EnvReader;

import static com.amazonaws.services.lambda.runtime.api.client.ReservedRuntimeEnvironmentVariables.*;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: prefer explicit imports

import static com.amazonaws.services.lambda.runtime.api.client.UserFault.filterStackTrace;
import static com.amazonaws.services.lambda.runtime.api.client.UserFault.makeUserFault;
import static com.amazonaws.services.lambda.runtime.api.client.UserFault.trace;
import static com.amazonaws.services.lambda.runtime.api.client.UserFault.*;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: prefer explicit imports

@smirnoal smirnoal merged commit 3175265 into main Apr 7, 2023
@smirnoal smirnoal deleted the formatting branch April 7, 2023 12:15
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.

3 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.