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

Bump minimum required version of Msft.Extensions.Logging to 3.1 #3196

Merged
merged 3 commits into from
Apr 15, 2022

Conversation

cijothomas
Copy link
Member

@cijothomas cijothomas commented Apr 15, 2022

2.1 has reached EOL last year...

@cijothomas cijothomas requested a review from a team April 15, 2022 20:21
Copy link
Member

@reyang reyang left a comment

Choose a reason for hiding this comment

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

Finally! 🍾

@reyang
Copy link
Member

reyang commented Apr 15, 2022

2.1 has reached EOF last year...

EOL

@codecov
Copy link

codecov bot commented Apr 15, 2022

Codecov Report

Merging #3196 (6626909) into main (1d16e0b) will decrease coverage by 0.01%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #3196      +/-   ##
==========================================
- Coverage   85.17%   85.16%   -0.02%     
==========================================
  Files         259      259              
  Lines        9345     9345              
==========================================
- Hits         7960     7959       -1     
- Misses       1385     1386       +1     
Impacted Files Coverage Δ
...emetry.Api/Internal/OpenTelemetryApiEventSource.cs 79.41% <0.00%> (-2.95%) ⬇️

@cijothomas cijothomas merged commit 2f8bcc4 into open-telemetry:main Apr 15, 2022
@cijothomas cijothomas deleted the cijothomas/bumpmelversion branch March 30, 2023 23:08
# 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.

2 participants