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

Fixed offset by one in AspNetRequestDurationLayoutRenderer #836

Merged
merged 1 commit into from
Jul 29, 2022

Conversation

snakefoot
Copy link
Contributor

No description provided.

@snakefoot snakefoot added this to the 5.1.1 milestone Jul 29, 2022
@snakefoot snakefoot added the bug label Jul 29, 2022
@codecov-commenter
Copy link

Codecov Report

Merging #836 (63ac506) into master (bcdfce5) will decrease coverage by 0%.
The diff coverage is 0%.

@@          Coverage Diff          @@
##           master   #836   +/-   ##
=====================================
- Coverage      69%    69%   -0%     
=====================================
  Files          63     63           
  Lines        1157   1157           
  Branches      286    286           
=====================================
- Hits          804    800    -4     
- Misses        229    234    +5     
+ Partials      124    123    -1     
Impacted Files Coverage Δ
...utRenderers/AspNetRequestDurationLayoutRenderer.cs 56% <0%> (-9%) ⬇️

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 bcdfce5...63ac506. Read the comment docs.

@sonarcloud
Copy link

sonarcloud bot commented Jul 29, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

100.0% 100.0% Coverage
0.0% 0.0% Duplication

@snakefoot snakefoot merged commit 499850e into NLog:master Jul 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants