Refactor: pass command kwargs to run

This allows to set configuration in instance variables (local)
instead of class variables (global).

Author: ysbaddaden

src/commands/check.cr

@@ -4,7 +4,7 @@ require "../versions"
 module Shards
   module Commands
     class Check < Command
-      def run(*args)
+      def run
         if has_dependencies?
           locks # ensures that lockfile exists
           verify(spec.dependencies)

src/commands/command.cr

@@ -22,10 +22,10 @@ module Shards
       @lockfile_path = File.join(@path, LOCK_FILENAME)
     end
 
-    abstract def run(*args)
+    abstract def run(*args, **kwargs)
 
-    def self.run(path, *args)
-      new(path).run(*args)
+    def self.run(path, *args, **kwargs)
+      new(path).run(*args, **kwargs)
     end
 
     def spec

src/commands/init.cr

@@ -4,7 +4,7 @@ require "ecr/macros"
 module Shards
   module Commands
     class Init < Command
-      def run(*args)
+      def run
         if File.exists?(shard_path)
           raise Error.new("#{SPEC_FILENAME} already exists")
         end

src/commands/install.cr

@@ -4,7 +4,7 @@ module Shards
   module Commands
     # OPTIMIZE: avoid updating GIT caches until required
     class Install < Command
-      def run(*args)
+      def run
         if lockfile?
           manager.locks = locks
           manager.resolve

src/commands/list.cr

@@ -3,11 +3,9 @@ require "./command"
 module Shards
   module Commands
     class List < Command
-      def self.run(path, @@tree = false)
-        super
-      end
+      @tree = false
 
-      def run(*args)
+      def run(@tree = false)
         return unless has_dependencies?
         puts "Shards installed:"
         list(spec.dependencies)
@@ -27,7 +25,7 @@ module Shards
           indent = "  " * level
           puts "#{indent}* #{_spec.name} (#{_spec.version})"
 
-          indent_level = @@tree ? level + 1 : level
+          indent_level = @tree ? level + 1 : level
           list(_spec.dependencies, indent_level)
         end
       end

src/commands/outdated.cr

@@ -3,14 +3,12 @@ require "./command"
 module Shards
   module Commands
     class Outdated < Command
+      @prereleases = false
+
       @up_to_date = true
       @output = IO::Memory.new
 
-      def self.run(path, @@prereleases = false)
-        super
-      end
-
-      def run(*args)
+      def run(@prereleases = false)
         return unless has_dependencies?
 
         if lockfile?
@@ -44,7 +42,7 @@ module Shards
         installed = _spec.version
 
         # already the latest version?
-        latest = Versions.sort(package.available_versions(@@prereleases)).first
+        latest = Versions.sort(package.available_versions(@prereleases)).first
         return if latest == installed
 
         @up_to_date = false
@@ -53,7 +51,7 @@ module Shards
         @output << " (installed: " << installed
 
         # is new version matching constraints available?
-        available = package.matching_versions(@@prereleases).first
+        available = package.matching_versions(@prereleases).first
         unless available == installed
           @output << ", available: " << available
         end

src/commands/prune.cr

@@ -5,7 +5,7 @@ require "../helpers/path"
 module Shards
   module Commands
     class Prune < Command
-      def run(*args)
+      def run
         return unless lockfile?
 
         Dir[File.join(Shards.install_path, "*")].each do |path|

src/commands/update.cr

@@ -4,7 +4,7 @@ module Shards
   module Commands
     class Update < Command
       # TODO: only update specified dependencies (ie. load locked versions, but don't enforce them)
-      def run(*args)
+      def run
         manager.resolve
 
         install(manager.packages)

src/commands/version.cr

@@ -3,12 +3,12 @@ require "./command"
 module Shards
   module Commands
     class Version < Command
-      def self.run(path, *args)
+      def self.run(path)
         path = lookup_path(path)
-        new(path).run(*args)
+        new(path).run
       end
 
-      def run(*args)
+      def run
         puts spec.version
       end