Clean up Build.cs

fluentassertions · Jan 7, 2023 · e46e75c · e46e75c
1 parent 37fbe54
commit e46e75c
Showing 1 changed file with 12 additions and 19 deletions.
diff --git a/Build/Build.cs b/Build/Build.cs
@@ -87,14 +87,14 @@ class Build : NukeBuild
             SemVer = GitVersion.SemVer;
             if (IsPullRequest)
             {
-                Serilog.Log.Information(
+                Information(
                     "Branch spec {branchspec} is a pull request. Adding build number {buildnumber}",
                     BranchSpec, BuildNumber);
 
                 SemVer = string.Join('.', GitVersion.SemVer.Split('.').Take(3).Union(new[] { BuildNumber }));
             }
 
-            Serilog.Log.Information("SemVer = {semver}", SemVer);
+            Information("SemVer = {semver}", SemVer);
         });
 
     bool IsPullRequest => BranchSpec != null && BranchSpec.Contains("pull", StringComparison.InvariantCultureIgnoreCase);
@@ -198,7 +198,7 @@ class Build : NukeBuild
                 .SetAssemblyFilters("+FluentAssertions"));
 
             string link = TestResultsDirectory / "reports" / "index.html";
-            Serilog.Log.Information($"Code coverage report: \x1b]8;;file://{link.Replace('\\', '/')}\x1b\\{link}\x1b]8;;\x1b\\");
+            Information($"Code coverage report: \x1b]8;;file://{link.Replace('\\', '/')}\x1b\\{link}\x1b]8;;\x1b\\");
         });
 
     Target TestFrameworks => _ => _
@@ -289,23 +289,16 @@ class Build : NukeBuild
     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();
-        }
-    }
+    string[] Changes =>
+        Repository.Diff
+            .Compare<TreeChanges>(TargetBranch, SourceBranch)
+            .Where(x => x.Exists)
+            .Select(x => x.Path)
+            .ToArray();
 
+    Repository Repository => new Repository(GitRepository.LocalDirectory);
+    Tree TargetBranch => Repository.Branches[PullRequestBase].Tip.Tree;
+    Tree SourceBranch => Repository.Branches[Repository.Head.FriendlyName].Tip.Tree;
     bool RunAllTargets => PullRequestBase == default;
 
     bool IsTag => BranchSpec != null && BranchSpec.Contains("refs/tags", StringComparison.InvariantCultureIgnoreCase);