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

feat: add population of the SourceLocation from context #721

Merged
merged 2 commits into from Oct 22, 2021

Conversation

minherz
Copy link
Contributor

@minherz minherz commented Oct 21, 2021

Instantiates a new object of the SourceLocation class from the current context by using the current thread's stack trace.
Add relevant unit testing methods to verify the behavior.

Fixes #686

@minherz minherz requested review from a team as code owners October 21, 2021 19:18
@product-auto-label product-auto-label bot added the api: logging Issues related to the googleapis/java-logging API. label Oct 21, 2021
@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label Oct 21, 2021
@minherz minherz merged commit 1fa3a6e into main Oct 22, 2021
@minherz minherz deleted the minherz/fix-686 branch October 22, 2021 06:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api: logging Issues related to the googleapis/java-logging API. cla: yes This human has signed the Contributor License Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

structured logging: add source location discovery
3 participants