Skip to content

Commit

Permalink
chore(deps): revert xunit.runner.visualstudio to pre v2.5 as some cov…
Browse files Browse the repository at this point in the history
…erage is missing because it is not compatible with eg. .NET Core App 3.1. Ensure build fails if NuGet restore fails as such.

This reverts commits:
- 8b70587
- 18601df
- bd5705b
- a89d9fb
  • Loading branch information
skwasjer committed Oct 17, 2023
1 parent 8b70587 commit fcc89b6
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 3 deletions.
1 change: 1 addition & 0 deletions Directory.Build.props
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,7 @@
<Nullable>enable</Nullable>
<ImplicitUsings>enable</ImplicitUsings>
<ContinuousIntegrationBuild Condition="'$(Configuration)'=='Release'">true</ContinuousIntegrationBuild>
<WarningsAsErrors>$(WarningsAsErrors);NU1605;NU1701</WarningsAsErrors>
</PropertyGroup>

<PropertyGroup>
Expand Down
2 changes: 1 addition & 1 deletion src/Directory.Build.targets
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
<Project>

<PropertyGroup>
<WarningsAsErrors>NU1605;CS1591</WarningsAsErrors>
<WarningsAsErrors>$(WarningsAsErrors);CS1591</WarningsAsErrors>
</PropertyGroup>

<PropertyGroup>
Expand Down
4 changes: 2 additions & 2 deletions test/Directory.Build.targets
Original file line number Diff line number Diff line change
Expand Up @@ -14,12 +14,12 @@
<ItemGroup>
<PackageReference Include="FluentAssertions" Version="6.12.0" />
<PackageReference Include="NSubstitute" Version="5.1.0" />
<PackageReference Include="xunit" Version="2.5.1" />
<PackageReference Include="xunit" Version="2.4.2" />
</ItemGroup>

<ItemGroup Condition="'$(IsTestProject)'=='true'">
<PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.7.2" />
<PackageReference Include="xunit.runner.visualstudio" Version="2.5.1">
<PackageReference Include="xunit.runner.visualstudio" Version="2.4.5">
<PrivateAssets>all</PrivateAssets>
<IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
</PackageReference>
Expand Down

0 comments on commit fcc89b6

Please sign in to comment.