Skip to content

Basic log scope support in ConsoleExporter - #1895

Merged
cijothomas merged 9 commits into
mainfrom
reyang/log-scope
Mar 11, 2021
Merged

Basic log scope support in ConsoleExporter#1895
cijothomas merged 9 commits into
mainfrom
reyang/log-scope

Conversation

@reyang

@reyang reyang commented Mar 10, 2021

Copy link
Copy Markdown
Member

Changes

Added log scope support to the ConsoleExporter

@reyang
reyang requested a review from a team March 10, 2021 05:40
@codecov

codecov Bot commented Mar 10, 2021

Copy link
Copy Markdown

Codecov Report

Merging #1895 (fe26965) into main (cee43b7) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #1895   +/-   ##
=======================================
  Coverage   83.35%   83.35%           
=======================================
  Files         188      188           
  Lines        6139     6139           
=======================================
  Hits         5117     5117           
  Misses       1022     1022           

Comment thread docs/logs/extending-the-sdk/MyExporter.cs Outdated
Comment thread src/OpenTelemetry.Exporter.Console/ConsoleLogRecordExporter.cs Outdated
@reyang
reyang force-pushed the reyang/log-scope branch from a57b881 to c2b0c32 Compare March 11, 2021 03:27
@cijothomas
cijothomas merged commit c404e19 into main Mar 11, 2021
@cijothomas
cijothomas deleted the reyang/log-scope branch March 11, 2021 22:19
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