Check incremental re-builds in CI (#123)

Author: jjonescz

.github/workflows/build.yml

@@ -27,7 +27,21 @@ jobs:
         run: dotnet workload install wasm-tools wasm-experimental
 
       - name: Build
-        run: dotnet build -warnaserror -p:TreatWarningsAsErrors=true
+        id: build
+        run: dotnet build -warnaserror -p:TreatWarningsAsErrors=true -bl:build.binlog
+
+      - name: Incremental re-build
+        run: dotnet build -warnaserror -p:TreatWarningsAsErrors=true -question -bl:question.binlog
 
       - name: Test
-        run: dotnet test --no-build
+        if: ${{ always() && steps.build.outcome == 'success' }}
+        run: dotnet test --no-build --report-xunit
+
+      - name: Upload logs
+        if: always()
+        uses: actions/upload-artifact@v6
+        with:
+          name: logs
+          path: |
+            *.binlog
+            **/TestResults/*.*

src/App/App.csproj

@@ -37,9 +37,14 @@
     <NpmInput Include="Npm\**" Exclude="Npm\node_modules\**;Npm\package-lock.json" />
     <NpmOutput Include="wwwroot\js\jslib.js" />
 
+    <!--
+      Ignore content generated by npm as it's not picked up by clean builds anyway,
+      only by rebuilds, but a rebuild immediately after a clean build wouldn't be incremental.
+    -->
+    <Content Remove="wwwroot\js\jslib.js*" />
+    <None Remove="wwwroot\js\jslib.js*" />
+
     <!-- Mark as Content so it's included in `service-worker-assets.js`. -->
-    <None Remove="@(NpmOutput)" />
-    <Content Remove="@(NpmOutput)" />
     <Content Include="@(NpmOutput)" />
   </ItemGroup>