Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

feat: DefaultTags for ASP.NET Core and M.E.Logging #268

Merged
merged 1 commit into from
Aug 25, 2019

Conversation

bruno-garcia
Copy link
Member

@bruno-garcia bruno-garcia commented Aug 23, 2019

Resolves #267

@bruno-garcia bruno-garcia force-pushed the feat/aspnetcore-and-mel-default-tags branch from a8038da to fd0616d Compare August 23, 2019 15:54
@bruno-garcia bruno-garcia force-pushed the feat/aspnetcore-and-mel-default-tags branch from fd0616d to 3c04b5d Compare August 23, 2019 15:57
@codecov-io
Copy link

codecov-io commented Aug 23, 2019

Codecov Report

Merging #268 into master will increase coverage by 0.02%.
The diff coverage is 100%.

Impacted file tree graph

@@           Coverage Diff            @@
##           master   #268      +/-   ##
========================================
+ Coverage   91.97%    92%   +0.02%     
========================================
  Files          81     81              
  Lines        2044   2050       +6     
========================================
+ Hits         1880   1886       +6     
  Misses        164    164
Impacted Files Coverage Δ
...tensions.Logging/SentryLoggingOptionsExtensions.cs 100% <100%> (ø) ⬆️
src/Sentry.AspNetCore/AspNetCoreEventProcessor.cs 100% <100%> (ø) ⬆️
.../Sentry.Extensions.Logging/SentryLoggingOptions.cs 100% <100%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f264f89...3c04b5d. Read the comment docs.

@bruno-garcia bruno-garcia merged commit 1040fe6 into master Aug 25, 2019
@bruno-garcia bruno-garcia deleted the feat/aspnetcore-and-mel-default-tags branch August 25, 2019 18:45
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

ASP.NET Core tags via appsettings.json
2 participants