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

Update net7.0 from main #3528

Merged
merged 4 commits into from
Aug 3, 2022
Merged

Update net7.0 from main #3528

merged 4 commits into from
Aug 3, 2022

Conversation

alanwest
Copy link
Member

@alanwest alanwest commented Aug 3, 2022

This is the first PR to net7.0 after I fixed it per this comment #3526 (comment)

This PR should also serve to run the CI and validate I didn't screw anything up when fixing things.

NOTE: This PR should be merged to net7.0 using a Merge commit. Merge commits may need to be enabled. I'll verify we can do this just for this branch rather than the whole repo.

@alanwest alanwest requested a review from a team August 3, 2022 19:37
@codecov
Copy link

codecov bot commented Aug 3, 2022

Codecov Report

Merging #3528 (25df7e8) into net7.0 (2b15b3d) will increase coverage by 0.41%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           net7.0    #3528      +/-   ##
==========================================
+ Coverage   86.47%   86.89%   +0.41%     
==========================================
  Files         275      275              
  Lines        9960     9948      -12     
==========================================
+ Hits         8613     8644      +31     
+ Misses       1347     1304      -43     
Impacted Files Coverage Δ
...tation.AspNetCore/Implementation/HttpInListener.cs 89.74% <ø> (-0.13%) ⬇️
...rometheusExporterMeterProviderBuilderExtensions.cs 100.00% <100.00%> (ø)
...etheus.HttpListener/Internal/PrometheusExporter.cs 100.00% <100.00%> (ø)
....Prometheus.HttpListener/PrometheusHttpListener.cs 82.66% <100.00%> (ø)
...theusHttpListenerMeterProviderBuilderExtensions.cs 75.00% <100.00%> (ø)
...tion.AspNetCore/TracerProviderBuilderExtensions.cs 100.00% <100.00%> (ø)
....Instrumentation.Http/HttpClientInstrumentation.cs 100.00% <100.00%> (ø)
...tp/Implementation/HttpHandlerDiagnosticListener.cs 73.49% <100.00%> (+0.86%) ⬆️
...umentation.Http/TracerProviderBuilderExtensions.cs 71.42% <100.00%> (-3.58%) ⬇️
src/OpenTelemetry/Logs/OpenTelemetryLogger.cs 86.66% <0.00%> (-2.23%) ⬇️
... and 14 more

This reverts commit e64015c, reversing
changes made to 5653454.
@alanwest alanwest merged commit 25df7e8 into net7.0 Aug 3, 2022
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.

5 participants