Skip to content

Commit

Permalink
chore: Remove NuGet.Frameworks dependency from test projects (#9276)
Browse files Browse the repository at this point in the history
* chore: fix NuGet.Frameworks package version

* chore: remove NuGet.Frameworks dependency from test projects

* chore: update test sdk version

* chore: remove NuGet.config file that is used for NuGet.Frameworks

---------

Co-authored-by: Yufei Huang <[email protected]>
  • Loading branch information
filzrev and yufeih committed Nov 16, 2023
1 parent 12a2fd2 commit 0a19218
Show file tree
Hide file tree
Showing 4 changed files with 1 addition and 63 deletions.
4 changes: 1 addition & 3 deletions Directory.Packages.props
Original file line number Diff line number Diff line change
Expand Up @@ -39,9 +39,7 @@
<PackageVersion Include="coverlet.collector" Version="6.0.0" />
<PackageVersion Include="FluentAssertions" Version="6.12.0" />
<PackageVersion Include="Magick.NET-Q16-AnyCPU" Version="13.4.0" />
<PackageVersion Include="Microsoft.NET.Test.Sdk" Version="17.8.0" />
<PackageVersion Include="NuGet.Frameworks" Version="6.8.0-rc.122" Condition="'$(TargetFramework)' != 'net8.0'" />
<PackageVersion Include="NuGet.Frameworks" Version="6.8.0-rc.122" Condition="'$(TargetFramework)' == 'net8.0'" />
<PackageVersion Include="Microsoft.NET.Test.Sdk" Version="17.9.0-preview-23531-01" />
<PackageVersion Include="Verify.DiffPlex" Version="2.3.0" />
<PackageVersion Include="Verify.Xunit" Version="22.4.1" />
<PackageVersion Include="xunit.runner.visualstudio" Version="2.5.3" />
Expand Down
8 changes: 0 additions & 8 deletions NuGet.config

This file was deleted.

10 changes: 0 additions & 10 deletions test/Directory.Build.props
Original file line number Diff line number Diff line change
Expand Up @@ -17,16 +17,6 @@
</PackageReference>
<PackageReference Include="FluentAssertions" />
<PackageReference Include="Microsoft.NET.Test.Sdk" />
<!--
The NuGet.Frameworks package is required to run csproj based tests targeting .NET 7.0.200.
Microsoft.NET.Test.Sdk bundles an old NuGet.Frameworks package to the output folder,
which takes predence over the one shipped with .NET SDK.
.NET 7.0.200 SDK depends on a newer NuGet.Frameworks package this is not availble on NuGet.org yet.
This only impact test since MSBuildLocator will resolve NuGet.Frameworks package from the appropreciate .NET SDK location.
-->
<PackageReference Include="NuGet.Frameworks" />
<PackageReference Include="xunit.runner.visualstudio" />
<PackageReference Include="xunit" />
</ItemGroup>
Expand Down
42 changes: 0 additions & 42 deletions test/docfx.Tests/ModuleInitializer.cs

This file was deleted.

0 comments on commit 0a19218

Please sign in to comment.