Renamed 'output' to 'outputs' where it makes sense

CHANGES.md

@@ -4,6 +4,9 @@
 
 * Tasks are mostly YAML serializable (procs can't be serialized)
 * Task class uses properties instead of instance variables
+* TaskManager is now a struct
+* Renamed argument `output` to `outputs` in `Task.initialize` where
+  it makes sense.
 
 ## Version v0.1.8
 

spec/croupier_spec.cr

@@ -44,7 +44,7 @@ def with_scenario(
       tasks.as_h.values.each do |t|
         Task.new(
           name: t["name"].to_s,
-          output: t["outputs"].as_a.map(&.to_s),
+          outputs: t["outputs"].as_a.map(&.to_s),
           inputs: t["inputs"].as_a.map(&.to_s),
           proc: _procs[t["procs"]],
           always_run: t["always_run"].as_bool,

src/croupier.cr

@@ -44,23 +44,22 @@ module Croupier
     # always_run is a boolean that tells croupier that the task is always
     #   stale regardless of its dependencies' state
     # FIXME: the id/name/output thing is confusing
-    # FIXME: rename output to outputs
     def initialize(
       name : String,
-      output : Array(String) = [] of String,
+      outputs : Array(String) = [] of String,
       inputs : Array(String) = [] of String,
       proc : TaskProc | Nil = nil,
       no_save : Bool = false,
       id : String | Nil = nil,
       always_run : Bool = false
     )
-      if !(inputs.to_set & output.to_set).empty?
+      if !(inputs.to_set & outputs.to_set).empty?
         raise "Cycle detected"
       end
       @always_run = always_run
       @name = name
       @procs << proc unless proc.nil?
-      @outputs = output.uniq
+      @outputs = outputs.uniq
       @id = id ? id : Digest::SHA1.hexdigest(@outputs.join(","))
       @inputs = inputs
       @no_save = no_save