-
Notifications
You must be signed in to change notification settings - Fork 90
docs: Implement Logger and Tracer part #27
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
Conversation
@PowerToolsLogging(logEvent = true) | ||
public APIGatewayProxyResponseEvent handleRequest(final APIGatewayProxyRequestEvent input, final Context context) { | ||
... | ||
PowerLogger.customKey("test", "willBeLogged"); |
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.
We will probably rename this api based on suggestion from heitor to make it explicit that it will append to the log. Just a note
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.
Looks good to me
Probably need to rebase with master :p |
Issue #, if available:
Description of changes:
Implement 2 new docs pages:
Checklist
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.