Optimize NUKE spell check

.github/workflows/build.yml

@@ -1,12 +1,6 @@
 name: build
 
-on:
-  pull_request:
-    paths-ignore:
-      - docs/**
-  push:
-    paths-ignore:
-      - docs/**
+on: [push, pull_request]
 
 jobs:
   build:
@@ -35,6 +29,7 @@ jobs:
       env:
         BranchSpec: ${{ github.ref }}
         BuildNumber: ${{ github.run_number }}
+        PullRequestBase: ${{ github.event.pull_request.base.ref }}
         ApiKey: ${{ secrets.NUGETAPIKEY }}
 
     - name: coveralls
@@ -71,3 +66,5 @@ jobs:
 
     - name: Run NUKE
       run: ./build.sh UnitTests
+      env:
+        BaseRef: ${{ github.event.pull_request.base.ref }}

.nuke/build.schema.json

@@ -66,6 +66,10 @@
             "type": "string"
           }
         },
+        "PullRequestBase": {
+          "type": "string",
+          "description": "The target branch for the pull request"
+        },
         "Root": {
           "type": "string",
           "description": "Root directory during build execution"

Build/Build.cs

@@ -1,8 +1,10 @@
 using System;
 using System.Collections.Generic;
 using System.Linq;
+using LibGit2Sharp;
 using Nuke.Common;
 using Nuke.Common.Execution;
+using Nuke.Common.Git;
 using Nuke.Common.IO;
 using Nuke.Common.ProjectModel;
 using Nuke.Common.Tooling;
@@ -28,14 +30,17 @@ class Build : NukeBuild
        - Microsoft VSCode           https://nuke.build/vscode
     */
 
-    public static int Main() => Execute<Build>(x => x.Push);
+    public static int Main() => Execute<Build>(x => x.SpellCheck, x => x.Push);
 
     [Parameter("A branch specification such as develop or refs/pull/1775/merge")]
     readonly string BranchSpec;
 
     [Parameter("An incrementing build number as provided by the build engine")]
     readonly string BuildNumber;
 
+    [Parameter("The target branch for the pull request")]
+    readonly string PullRequestBase;
+
     [Parameter("The key to push to Nuget")]
     readonly string ApiKey;
 
@@ -45,6 +50,9 @@ class Build : NukeBuild
     [GitVersion(Framework = "net6.0")]
     readonly GitVersion GitVersion;
 
+    [GitRepository]
+    readonly GitRepository GitRepository;
+
     [PackageExecutable("nspec", "NSpecRunner.exe", Version = "3.1.0")]
     Tool NSpec3;
 
@@ -65,13 +73,15 @@ class Build : NukeBuild
     string YarnCli => ToolPathResolver.GetPackageExecutable("Yarn.MSBuild", "yarn.js", "1.22.19");
 
     Target Clean => _ => _
+        .OnlyWhenDynamic(() => RunAllTargets || HasSourceChanges)
         .Executes(() =>
         {
             EnsureCleanDirectory(ArtifactsDirectory);
             EnsureCleanDirectory(TestResultsDirectory);
         });
 
     Target CalculateNugetVersion => _ => _
+        .OnlyWhenDynamic(() => RunAllTargets || HasSourceChanges)
         .Executes(() =>
         {
             SemVer = GitVersion.SemVer;
@@ -91,6 +101,7 @@ class Build : NukeBuild
 
     Target Restore => _ => _
         .DependsOn(Clean)
+        .OnlyWhenDynamic(() => RunAllTargets || HasSourceChanges)
         .Executes(() =>
         {
             DotNetRestore(s => s
@@ -101,6 +112,7 @@ class Build : NukeBuild
 
     Target Compile => _ => _
         .DependsOn(Restore)
+        .OnlyWhenDynamic(() => RunAllTargets || HasSourceChanges)
         .Executes(() =>
         {
             DotNetBuild(s => s
@@ -115,6 +127,7 @@ class Build : NukeBuild
 
     Target ApiChecks => _ => _
         .DependsOn(Compile)
+        .OnlyWhenDynamic(() => RunAllTargets || HasSourceChanges)
         .Executes(() =>
         {
             DotNetTest(s => s
@@ -126,6 +139,7 @@ class Build : NukeBuild
 
     Target UnitTests => _ => _
         .DependsOn(Compile)
+        .OnlyWhenDynamic(() => RunAllTargets || HasSourceChanges)
         .Executes(() =>
         {
             Project[] projects = new[]
@@ -172,6 +186,7 @@ class Build : NukeBuild
     Target CodeCoverage => _ => _
         .DependsOn(TestFrameworks)
         .DependsOn(UnitTests)
+        .OnlyWhenDynamic(() => RunAllTargets || HasSourceChanges)
         .Executes(() =>
         {
             ReportGenerator(s => s
@@ -188,6 +203,7 @@ class Build : NukeBuild
 
     Target TestFrameworks => _ => _
         .DependsOn(Compile)
+        .OnlyWhenDynamic(() => RunAllTargets || HasSourceChanges)
         .Executes(() =>
         {
             var testCombinations =
@@ -227,6 +243,7 @@ class Build : NukeBuild
         .DependsOn(UnitTests)
         .DependsOn(CodeCoverage)
         .DependsOn(CalculateNugetVersion)
+        .OnlyWhenDynamic(() => RunAllTargets || HasSourceChanges)
         .Executes(() =>
         {
             DotNetPack(s => s
@@ -258,11 +275,38 @@ class Build : NukeBuild
         });
 
     Target SpellCheck => _ => _
+        .OnlyWhenDynamic(() => RunAllTargets || HasDocumentationChanges)
         .Executes(() =>
         {
             Node($"{YarnCli} install --silent", workingDirectory: RootDirectory);
             Node($"{YarnCli} --silent run cspell --no-summary", workingDirectory: RootDirectory,
                 customLogger: (_, msg) => Error(msg));
         });
+
+    bool HasDocumentationChanges =>
+        Changes.Any(x => x.StartsWith("docs"));
+
+    bool HasSourceChanges =>
+        Changes.Any(x => !x.StartsWith("docs"));
+
+    string[] Changes
+    {
+        get
+        {
+            using var repo = new Repository(GitRepository.LocalDirectory);
+
+            Tree targetBranch = repo.Branches[PullRequestBase].Tip.Tree;
+            Tree sourceBranch = repo.Branches[repo.Head.FriendlyName].Tip.Tree;
+
+            return repo.Diff
+                .Compare<TreeChanges>(targetBranch, sourceBranch)
+                .Where(x => x.Exists)
+                .Select(x => x.Path)
+                .ToArray();
+        }
+    }
+
+    bool RunAllTargets => PullRequestBase == default;
+
     bool IsTag => BranchSpec != null && BranchSpec.Contains("refs/tags", StringComparison.InvariantCultureIgnoreCase);
 }

Build/_build.csproj

@@ -22,6 +22,7 @@
     <PackageDownload Include="ReportGenerator" Version="[5.1.13]" />
     <PackageDownload Include="xunit.runner.console" Version="[2.4.2]" />
     <PackageDownload Include="Node.js.redist" Version="[16.17.1]" />
+    <PackageReference Include="LibGit2Sharp" Version="0.27.0-preview-0182" />
     <PackageReference Include="Nuke.Common" Version="6.3.0" />
     <PackageDownload Include="Yarn.MSBuild" Version="[1.22.19]" />
   </ItemGroup>

package.json

@@ -1,7 +1,7 @@
 {
     "version": "1.0.0",
     "scripts": {
-        "cspell": "cspell --config ./cSpell.json ./**/*.md --no-progress"
+        "cspell": "cspell --config ./cSpell.json ./docs/**/*.md --no-progress"
     },
     "dependencies": {
         "cspell": "^6.18.1"