Skip to content

doc(java): Global tags/extra with logback appender encoder - #1379

Merged
bruno-garcia merged 7 commits into
getsentry:masterfrom
jeacott1:jeacott1-logback-global
Dec 3, 2019
Merged

doc(java): Global tags/extra with logback appender encoder#1379
bruno-garcia merged 7 commits into
getsentry:masterfrom
jeacott1:jeacott1-logback-global

Conversation

@jeacott1

Copy link
Copy Markdown
Contributor

adds documentation for global tags and extras using the logback appender.

I'm not sure what the target version is likely to be however? unlikely to remain 1.7.27

@bruno-garcia bruno-garcia left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Small changes suggested.
Other than that LGTM, @MimiDumpling will give the final approval before we merge/

Comment thread src/collections/_documentation/clients/java/integrations.md
Comment thread src/collections/_documentation/clients/java/integrations.md Outdated
Comment thread src/collections/_documentation/clients/java/integrations.md Outdated
@bruno-garcia

Copy link
Copy Markdown
Member

@jeacott1 Thanks for this btw, awesome work!

@bruno-garcia bruno-garcia changed the title Update integrations.md doc(java): Global tags/extra with logback appender encoder Nov 29, 2019
@dashed
dashed requested a review from MimiDumpling November 29, 2019 02:17
jeacott1 and others added 4 commits November 29, 2019 13:03
Co-Authored-By: Bruno Garcia <bruno@brunogarcia.com>
Co-Authored-By: Bruno Garcia <bruno@brunogarcia.com>
Co-Authored-By: Bruno Garcia <bruno@brunogarcia.com>

@MimiDumpling MimiDumpling left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Just a couple suggested edits. 😸

Comment thread src/collections/_documentation/clients/java/integrations.md Outdated
Comment thread src/collections/_documentation/clients/java/integrations.md Outdated
jeacott1 and others added 2 commits December 3, 2019 10:13
Co-Authored-By: Tien "Mimi" Nguyen <tienmiminguyen@gmail.com>
Co-Authored-By: Tien "Mimi" Nguyen <tienmiminguyen@gmail.com>

@bruno-garcia bruno-garcia left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks @jeacott1 !

@bruno-garcia
bruno-garcia merged commit c0ee769 into getsentry:master Dec 3, 2019
@github-actions github-actions Bot locked and limited conversation to collaborators Dec 16, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants