Skip to content

Commit

Permalink
Remove unnecessary explicit test dependency
Browse files Browse the repository at this point in the history
We don't need a dependency on Microsoft.Extensions.Configuration as
we already get that indirectly through
Microsoft.Extensions.Configuration.Json.
Having both dependencies means that we always need to update both
together. This problem goes away if we simply remove the dependency on
Microsoft.Extensions.Configuration.
  • Loading branch information
FlorianHockmann committed Nov 15, 2023
1 parent 62d6951 commit 28294bb
Showing 1 changed file with 0 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,6 @@
<PackageReference Include="NSubstitute" Version="5.1.0" />
<PackageReference Include="xunit.runner.visualstudio" Version="2.5.3" />
<PackageReference Include="xunit" Version="2.5.3" />
<PackageReference Include="Microsoft.Extensions.Configuration" Version="7.0.0" />
<PackageReference Include="Microsoft.Extensions.Configuration.Json" Version="8.0.0" />
</ItemGroup>
<ItemGroup>
Expand Down

0 comments on commit 28294bb

Please sign in to comment.