Better dependency order in FAKE

vbfox · Sep 9, 2018 · 2458f16 · 2458f16
1 parent e9acbba
commit 2458f16
Showing 1 changed file with 12 additions and 12 deletions.
diff --git a/src/BlackFox.MasterOfFoo.Build/Tasks.fs b/src/BlackFox.MasterOfFoo.Build/Tasks.fs
@@ -130,7 +130,18 @@ let createAndGetDefault () =
         Trace.publish ImportData.BuildArtifact zipFile
     }
 
-    let gitHubRelease = BuildTask.create "GitHubRelease" [zip] {
+    let gitRelease = BuildTask.create "GitRelease" [nuget.IfNeeded] {
+        let remote =
+            Git.CommandHelper.getGitResult "" "remote -v"
+            |> Seq.filter (fun (s: string) -> s.EndsWith("(push)"))
+            |> Seq.tryFind (fun (s: string) -> s.Contains(gitOwner + "/" + gitName))
+            |> function None -> gitHome + "/" + gitName | Some (s: string) -> s.Split().[0]
+
+        Git.Branches.tag "" release.NugetVersion
+        Git.Branches.pushTag "" remote release.NugetVersion
+    }
+
+    let gitHubRelease = BuildTask.create "GitHubRelease" [zip; gitRelease.IfNeeded] {
         let user =
             match Environment.environVarOrNone "github-user" with
             | Some s -> s
@@ -153,17 +164,6 @@ let createAndGetDefault () =
         |> Async.RunSynchronously
     }
 
-    let gitRelease = BuildTask.create "GitRelease" [] {
-        let remote =
-            Git.CommandHelper.getGitResult "" "remote -v"
-            |> Seq.filter (fun (s: string) -> s.EndsWith("(push)"))
-            |> Seq.tryFind (fun (s: string) -> s.Contains(gitOwner + "/" + gitName))
-            |> function None -> gitHome + "/" + gitName | Some (s: string) -> s.Split().[0]
-
-        Git.Branches.tag "" release.NugetVersion
-        Git.Branches.pushTag "" remote release.NugetVersion
-    }
-
     let _releaseTask = BuildTask.createEmpty "Release" [clean; gitRelease; gitHubRelease; publishNuget]
     let _ciTask = BuildTask.createEmpty "CI" [clean; runTests; zip; nuget]