Merge pull request #37 from MineralsCloud:shouldrun

Change `prepare!` & `shouldrun` for `AbstractJob` instead of `Executor`s

EasyJobsBase/src/run.jl

@@ -119,21 +119,21 @@ function _run!(job::AbstractJob)  # Do not export!
     return job
 end
 
-prepare!(::Executor) = nothing  # No op
-function prepare!(exec::Executor{ArgDependentJob})
+prepare!(::AbstractJob) = nothing  # No op
+function prepare!(job::ArgDependentJob)
     # Use previous results as arguments
-    args = if countparents(exec.job) == 1
-        something(getresult(only(eachparent(exec.job))))
+    args = if countparents(job) == 1
+        something(getresult(only(eachparent(job))))
     else  # > 1
-        Set(something(getresult(parent)) for parent in eachparent(exec.job))
+        Set(something(getresult(parent)) for parent in eachparent(job))
     end
-    setargs!(exec.job.core, args)
+    setargs!(job.core, args)
     return nothing
 end
 
-shouldrun(::Executor) = true
-shouldrun(exec::Executor{ConditionalJob}) =
-    countparents(exec.job) >= 1 && all(issucceeded(parent) for parent in exec.job.parents)
+shouldrun(::AbstractJob) = true
+shouldrun(job::ConditionalJob) =
+    countparents(job) >= 1 && all(issucceeded(parent) for parent in eachparent(job))
 
 """
     kill!(exec::Executor)