Skip to content

Conversation

@iliar-turdushev
Copy link
Contributor

@iliar-turdushev iliar-turdushev commented Nov 2, 2023

According to OTel semantic conventions we should use value close instead of closed.

image

Microsoft Reviewers: Open in CodeFlow

@dotnet-comment-bot
Copy link
Collaborator

🎉 Good job! The coverage increased 🎉
Update MinCodeCoverage in the project files.

Project Expected Actual
Microsoft.Extensions.Telemetry 92 93
Microsoft.AspNetCore.Diagnostics.Middleware 99 100

Full code coverage report: https://dev.azure.com/dnceng-public/public/_build/results?buildId=458256&view=codecoverage-tab

@danmoseley
Copy link
Member

Coverage increased?? 😀

Copy link
Contributor

@xakep139 xakep139 left a comment

Choose a reason for hiding this comment

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

Good catch! Thanks for fixing this one!

@geeknoid geeknoid merged commit b314c00 into release/8.0 Nov 2, 2023
@geeknoid geeknoid deleted the iliarturdu/fix-network-state-dim branch November 2, 2023 17:21
@ghost ghost removed the work in progress 🚧 label Nov 2, 2023
@github-actions github-actions bot locked and limited conversation to collaborators Dec 3, 2023
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.

6 participants