Skip to content

Commit dc0972c

Browse files
committed
final approach: there could be dependencies without targets such as AlwaysRerun
1 parent f3b3cb5 commit dc0972c

File tree

1 file changed

+4
-3
lines changed

1 file changed

+4
-3
lines changed

core/DependencyAnalysis.fs

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -106,10 +106,11 @@ let getChangeReasons ctx getTargetDeps target =
106106

107107
// gets task duration and list of targets it depends on. No clue why one method does both.
108108
let getDurationDeps ctx getDeps t =
109-
let deps = getDeps t |> List.collect (function |Depends t |DependsMissingTarget t -> [t] | _ -> [])
110-
match deps with
109+
match getDeps t with
111110
| [] -> 0<ms>, []
112-
| _ -> (getExecTime ctx t, deps)
111+
| deps ->
112+
let targets = deps |> List.collect (function |Depends t |DependsMissingTarget t -> [t] | _ -> [])
113+
(getExecTime ctx t, targets)
113114
// |> fun (tt,dd) ->
114115
// printfn "For task %A duration:%A deps:%A" t tt dd
115116
// (tt,dd)

0 commit comments

Comments
 (0)