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.

Sign up for GitHub

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

WIP fix: dispose IServiceProvider in MicrosoftDependencyResolver #1170

Open
wants to merge 13 commits into
base: main
Choose a base branch
from

Merge branch 'main' into dispose-microsoft-extensions-types

67b3cab
Select commit
Loading
Failed to load commit list.
Open

WIP fix: dispose IServiceProvider in MicrosoftDependencyResolver #1170

Merge branch 'main' into dispose-microsoft-extensions-types
67b3cab
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch failed Dec 30, 2024 in 0s

23.07% of diff hit (target 71.87%)

View this Pull Request on Codecov

23.07% of diff hit (target 71.87%)

Annotations

Check warning on line 65 in src/Splat.Microsoft.Extensions.DependencyInjection/MicrosoftDependencyResolver.cs

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/Splat.Microsoft.Extensions.DependencyInjection/MicrosoftDependencyResolver.cs#L65 

Added line #L65 was not covered by tests

Check warning on line 70 in src/Splat.Microsoft.Extensions.DependencyInjection/MicrosoftDependencyResolver.cs

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/Splat.Microsoft.Extensions.DependencyInjection/MicrosoftDependencyResolver.cs#L70 

Added line #L70 was not covered by tests

Check warning on line 73 in src/Splat.Microsoft.Extensions.DependencyInjection/MicrosoftDependencyResolver.cs

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/Splat.Microsoft.Extensions.DependencyInjection/MicrosoftDependencyResolver.cs#L73 

Added line #L73 was not covered by tests

Check warning on line 78 in src/Splat.Microsoft.Extensions.DependencyInjection/MicrosoftDependencyResolver.cs

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/Splat.Microsoft.Extensions.DependencyInjection/MicrosoftDependencyResolver.cs#L77-L78 

Added lines #L77 - L78 were not covered by tests

Check warning on line 83 in src/Splat.Microsoft.Extensions.DependencyInjection/MicrosoftDependencyResolver.cs

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/Splat.Microsoft.Extensions.DependencyInjection/MicrosoftDependencyResolver.cs#L81-L83 

Added lines #L81 - L83 were not covered by tests

Check warning on line 106 in src/Splat.Microsoft.Extensions.DependencyInjection/MicrosoftDependencyResolver.cs

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/Splat.Microsoft.Extensions.DependencyInjection/MicrosoftDependencyResolver.cs#L106 

Added line #L106 was not covered by tests

Check warning on line 247 in src/Splat.Microsoft.Extensions.DependencyInjection/MicrosoftDependencyResolver.cs

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/Splat.Microsoft.Extensions.DependencyInjection/MicrosoftDependencyResolver.cs#L247 

Added line #L247 was not covered by tests

Check warning on line 301 in src/Splat.Microsoft.Extensions.DependencyInjection/MicrosoftDependencyResolver.cs

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/Splat.Microsoft.Extensions.DependencyInjection/MicrosoftDependencyResolver.cs#L300-L301 

Added lines #L300 - L301 were not covered by tests

Check warning on line 303 in src/Splat.Microsoft.Extensions.DependencyInjection/MicrosoftDependencyResolver.cs

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/Splat.Microsoft.Extensions.DependencyInjection/MicrosoftDependencyResolver.cs#L303 

Added line #L303 was not covered by tests

Check warning on line 320 in src/Splat.Microsoft.Extensions.DependencyInjection/MicrosoftDependencyResolver.cs

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/Splat.Microsoft.Extensions.DependencyInjection/MicrosoftDependencyResolver.cs#L320 

Added line #L320 was not covered by tests

Check warning on line 322 in src/Splat.Microsoft.Extensions.DependencyInjection/MicrosoftDependencyResolver.cs

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/Splat.Microsoft.Extensions.DependencyInjection/MicrosoftDependencyResolver.cs#L322 

Added line #L322 was not covered by tests
View more details on Codecov