Merge pull request #883 from Romfos/doc1

Migrate documentation validation from build.fsproj to Roslyn code generator

Author: dtchepak

.github/workflows/test.yml

@@ -33,22 +33,6 @@ jobs:
       - name: Test
         run: dotnet test -f ${{ matrix.framework }} --no-build --no-restore
 
-  test-documentation:
-    runs-on: windows-latest
-    steps:
-      - name: Checkout
-        uses: actions/checkout@v4
-
-      - name: Setup .NET
-        uses: actions/setup-dotnet@v4
-        with:
-          dotnet-version: |
-            9.0.x
-            8.0.x
-
-      - name: Build documentation
-        run: dotnet run --project 'build/build.fsproj'
-
   format-verify:
     runs-on: ubuntu-latest
     steps:

CHANGELOG.md

@@ -9,6 +9,7 @@
 * [UPDATE] Migrate documentation to docfx platform. https://github.com/dotnet/docfx
 * [UPDATE][BREAKING] Nullability is enabled for public api for .NET Core TFMs
 * [UPDATE] Migrate to slnx format for solution file
+* [UPDATE] Migrate documentation validation from build.fsproj to Roslyn code generator
 
 ### 5.3.0 (October 2024)
 

Directory.Build.props

@@ -9,8 +9,4 @@
     <ImplicitUsings>enable</ImplicitUsings>
   </PropertyGroup>
 
-  <PropertyGroup>
-    <OutputPath>$(MSBuildThisFileDirectory)\bin\$(Configuration)\$(MSBuildProjectName)\</OutputPath>
-  </PropertyGroup>
-
 </Project>

NSubstitute.slnx

@@ -1,8 +1,4 @@
 <Solution>
-  <Folder Name="/build/">
-    <File Path="build/build.fs" />
-    <File Path="build/ExtractDocs.fs" />
-  </Folder>
   <Folder Name="/Solution Items/">
     <File Path=".editorconfig" />
     <File Path="BreakingChanges.md" />
@@ -16,5 +12,7 @@
   <Folder Name="/tests/">
     <Project Path="tests/NSubstitute.Acceptance.Specs/NSubstitute.Acceptance.Specs.csproj" />
     <Project Path="tests/NSubstitute.Benchmarks/NSubstitute.Benchmarks.csproj" />
+    <Project Path="tests/NSubstitute.Documentation.Tests.Generator/NSubstitute.Documentation.Tests.Generator.csproj" Id="d031abb6-0702-4eee-89f6-dd3e9af0573b" />
+    <Project Path="tests/NSubstitute.Documentation.Tests/NSubstitute.Documentation.Tests.csproj" Id="ed043d85-3df4-4878-ab35-ce5e533f1a30" />
   </Folder>
 </Solution>

build/ExtractDocs.fs

@@ -7,7 +7,7 @@
     <PublicSign Condition=" '$(OS)' != 'Windows_NT' ">true</PublicSign>
     <AssemblyOriginatorKeyFile>nsubstitute.snk</AssemblyOriginatorKeyFile>
     <DelaySign>false</DelaySign>
-    <DocumentationFile>..\..\bin\$(Configuration)\NSubstitute\$(TargetFramework)\NSubstitute.xml</DocumentationFile>
+    <GenerateDocumentationFile>true</GenerateDocumentationFile>
     <NoWarn>$(NoWarn);1591</NoWarn>
   </PropertyGroup>
 

build/build.fs

@@ -6,9 +6,9 @@
 
   <ItemGroup>
     <PackageReference Include="Microsoft.Extensions.Logging.Abstractions" Version="9.0.0" />
-    <PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.12.0" />
-    <PackageReference Include="NUnit" Version="4.2.2" />
-    <PackageReference Include="NUnit3TestAdapter" Version="4.6.0" />
+    <PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.14.0" />
+    <PackageReference Include="NUnit" Version="4.3.2" />
+    <PackageReference Include="NUnit3TestAdapter" Version="5.0.0" />
   </ItemGroup>
 
   <ItemGroup Condition="'$(TargetFramework)' == 'net462'">