sumodev 0.8.2 → 0.9.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 4a23d3eae495a82fb860954a128e8920f7dddcf6
-  data.tar.gz: 7b6f8c4b30b588790c35777bf066a7a7c305c769
+  metadata.gz: cb24a085590f4899c56883f060568e4912f323b8
+  data.tar.gz: d3a39b2e7bba8537d7f91c12974d968a094bd042
 SHA512:
-  metadata.gz: e6484a6d9e1994cd7db90c89898afdad12e8bdd4911df2a2bf86d7dc54af03831da3ab71a5022d3c1b64f2d96bc45209435ac4250f7f0ea24156998c58ab4fc8
-  data.tar.gz: f6fb989527b0d22add31b643df2b1372903b5568464ef5a0a3e1c593ead4b09b752ba4cf4cf76a8e5553c68352385043a6c6a9a59fe4fda386db7bde9279c9ec
+  metadata.gz: 813b45fb04576aa3eae3b1eaecabfcfc36229c5d5f005dfd88e11cc50a41ba31ec43848a0eb5de35bb18f0d7f8d69246f899e6f0a712214b25197d08d313e5e2
+  data.tar.gz: 76eae506592d22db3066d9f0ae71c29f3602ddff29926a8f808915d5d48e60810b1076153a5d41bd186e870a259ca96c2aeaef79c06b59d4aac54e5b13514ebe

data/lib/sumodev/commands/hooks.rb

@@ -0,0 +1,326 @@
+require 'sumodev/command'
+
+class Sumodev::Commands::Hooks < Sumodev::Command
+  namespace :hooks
+
+  no_commands do
+    def find_files_by_extension(files, extension)
+      files.select{ |i| i[/\.#{extension}$/] }
+    end
+
+    # Build a hash that we will use for errors or warnings.
+    def build_error_or_warning(type, line, message)
+      {
+        :type => type,
+        :line => line,
+        :message => message
+      }
+    end
+
+    def output_before_check(message)
+      # this has to end with a space so our next output is appended
+      say("--> #{message} ")
+    end
+
+    def check_files(files, methods)
+      errors_and_warnings = []
+
+      methods.each do |method|
+        has_warnings = false
+        has_errors = false
+        output_before_check(method[:human_readable_message])
+
+        files.each do |file|
+          case method[:command].class.to_s
+          when "String"
+            # if the command is a string we will run it on the CLI
+            command = "#{method[:command]}" % {:file => file}
+            IO.popen(command) do |f|
+              f.read.split("\n").each do |line|
+                error_or_warning = method[:line_handler].call(line)
+                unless error_or_warning.nil?
+                  if 'error' == error_or_warning[:type]
+                    has_errors = true
+                  elsif 'warning' == error_or_warning[:type]
+                    has_warnings = true
+                  end
+                  error_or_warning[:file] = file
+                  errors_and_warnings << error_or_warning
+                end
+              end
+            end
+          when "Proc"
+            # a method is provided so call it.
+            error_or_warning = method[:command].call(file)
+          else
+            raise "Unknown type"
+          end
+        end
+
+        output_result_after_check(
+          method[:human_readable_message],
+          has_errors,
+          has_warnings
+        )
+      end
+
+      errors_and_warnings
+    end
+
+    def check_css_files(files)
+      methods = [
+        {
+          :human_readable_message => "Checking for CSS-lint errors",
+          :command => "csslint --format=compact %{file}",
+          :line_handler => lambda { |line|
+            if line.include? "Warning - "
+              return build_error_or_warning(
+                'warning',
+                line.scan(/line\ ([0-9*])/i)[0][0],
+                line.scan(/Warning - (.*)/)[0][0]
+              )
+            end
+            if line.include? "Error - "
+              return build_error_or_warning(
+                'error',
+                line.scan(/line\ ([0-9*])/i)[0][0],
+                line.scan(/Error - (.*)/)[0][0]
+              )
+            end
+          }
+        }
+      ]
+
+      check_files(files, methods)
+    end
+
+    def check_scss_files(files)
+      methods = [
+        {
+          :human_readable_message => "Checking SCSS lint errors",
+          :command => "scss-lint %{file}",
+          :line_handler => lambda { |line|
+            if line.include? "[W] "
+              return build_error_or_warning(
+                'warning',
+                line.scan(/\:([0-9*])/i)[0][0],
+                line.scan(/\[W\] (.*)/)[0][0]
+              )
+            end
+            if line.include? "[E] "
+              return build_error_or_warning(
+                'error',
+                line.scan(/\:([0-9*])/i)[0][0],
+                line.scan(/\[E\] (.*)/)[0][0]
+              )
+            end
+          }
+        }
+      ]
+
+      check_files(files, methods)
+    end
+
+    def check_coffee_files(files)
+      methods = [
+        {
+          :human_readable_message => "Checking for keywords in Coffee-files",
+          :command => "egrep -no 'console\.' %{file}",
+          :line_handler => lambda { |line|
+            return build_error_or_warning(
+              'warning',
+              line.scan(/([0-9]*):/)[0][0],
+              "Keyword found: " + line.scan(/[0-9]*:(.*)/)[0][0]
+            )
+          }
+        }
+      ]
+
+      check_files(files, methods)
+    end
+
+    def check_js_files(files)
+      methods = [
+        {
+          :human_readable_message => "Checking for js errors",
+          :command => "esvalidate --format=junit %{file}",
+          :line_handler => lambda { |line|
+            if line.include? "<warning "
+              return build_error_or_warning(
+                'warning',
+                line.scan(/Line ([0-9]*):/)[0][0],
+                line.scan(/>Line.*: (.*)\(/)[0][0]
+              )
+            end
+            if line.include? "<error "
+              return build_error_or_warning(
+                'error',
+                line.scan(/Line ([0-9]*):/)[0][0],
+                line.scan(/>Line.*: (.*)\(/)[0][0]
+              )
+            end
+          }
+        },
+        {
+          :human_readable_message => "Checking for keywords in JS-files",
+          :command => "egrep -no 'console\.' %{file}",
+          :line_handler => lambda { |line|
+            return build_error_or_warning(
+              'warning',
+              line.scan(/([0-9]*):/)[0][0],
+              "Keyword found: " + line.scan(/[0-9]*:(.*)/)[0][0]
+            )
+          }
+        }
+      ]
+
+      check_files(files, methods)
+    end
+
+    def check_php_files(files)
+      methods = [
+        {
+          :human_readable_message => "Checking for PHP syntax errors",
+          :command => "php -l %{file} 2>&1",
+          :line_handler => lambda { |line|
+            if line.include? "PHP Parse error: "
+              return build_error_or_warning(
+                'error',
+                line.scan(/line ([0-9]*)$/)[0][0],
+                line.scan(/:  (.*) in/)[0][0]
+              )
+            end
+          }
+        },
+        {
+          :human_readable_message => "Checking for PHP Coding standards errors",
+          :command => "phpcs --standard=PSR2 --extensions=php --encoding=utf8 --report=xml %{file}",
+          :line_handler => lambda { |line|
+            if line.include? "<warning "
+              return build_error_or_warning(
+                'warning',
+                line.scan(/line="([0-9]*)"/)[0][0],
+                line.scan(/>(.*)<\/warning>/)[0][0]
+              )
+            end
+            if line.include? "<error "
+              return build_error_or_warning(
+                'error',
+                line.scan(/line="([0-9]*)"/)[0][0],
+                line.scan(/>(.*)<\/error>/)[0][0]
+              )
+            end
+          }
+        },
+        {
+          :human_readable_message => "Checking for keywords in PHP-files",
+          :command => "egrep -no '(dump\\(|var_dump\\(|print_r\\(|exit|die|Zend_Debug::)' %{file}",
+          :line_handler => lambda { |line|
+            return build_error_or_warning(
+              'warning',
+              line.scan(/([0-9]*):/)[0][0],
+              "Keyword found: " + line.scan(/[0-9]*:(.*)/)[0][0]
+            )
+          }
+        }
+      ]
+
+      check_files(files, methods)
+    end
+
+    def output_result_after_check(message, has_errors, has_warnings)
+      # as we append 7 characters ourself we should decrease the allowed maximum width.
+      say(".".rjust(72 - 7 - message.length, ".") + " ")
+      if has_errors
+        say("✘", :red)
+      elsif has_warnings
+        say("⚠", :yellow)
+      else
+        say("✔", :green)
+      end
+    end
+
+    def output_results(errors_and_warnings)
+      puts "\n"
+
+      if errors_and_warnings.empty?
+        say("Atta boy, no errors! 🍺", :green)
+        return
+      end
+
+      # split into seperate arrays
+      errors = []
+      warnings = []
+      errors_and_warnings.each do |group,items|
+        items.each do |error_or_warning|
+          if error_or_warning[:type] == "error"
+            errors << error_or_warning
+          end
+          if error_or_warning[:type] == "warning"
+            warnings << error_or_warning
+          end
+        end
+      end
+
+      # some encouragement
+      if errors.empty? && !warnings.empty?
+        say("⚠ Yeah, only warnings!", :yellow)
+      end
+      if !errors.empty?
+        say("✘ Woops, errors! Check them below.", :red)
+      end
+
+      # output the warnings and errors
+      if !warnings.empty?
+        puts "\nWarnings\n--------\n"
+        warnings.each do |warning|
+          puts "• Warning: %{file}:%{line} - %{message}" % warning
+        end
+      end
+
+      if !errors.empty?
+        puts "\nErrors\n------\n"
+        errors.each do |error|
+          puts "• Error: %{file}:%{line} - %{message}" % error
+        end
+      end
+    end
+  end
+
+  desc 'check', "check the files"
+  def check(*files)
+    errors_and_warnings = {}
+
+    # Check the CSS files
+    css_files = find_files_by_extension(files, 'css')
+    if css_files.any?
+      errors_and_warnings['CSS'] = check_css_files(css_files)
+    end
+
+    # Check the SCSS files
+    scss_files = find_files_by_extension(files, 's(c|a)ss')
+    if scss_files.any?
+      errors_and_warnings['SCSS'] = check_scss_files(scss_files)
+    end
+
+    # Check the Coffee files
+    coffee_files = find_files_by_extension(files, 'coffee')
+    if coffee_files.any?
+      errors_and_warnings['Coffee'] = check_coffee_files(coffee_files)
+    end
+
+    # Check the JS files
+    js_files = find_files_by_extension(files, 'js')
+    if js_files.any?
+      errors_and_warnings['JS'] = check_js_files(js_files)
+    end
+
+    # Check the php files
+    php_files = find_files_by_extension(files, 'php')
+    if php_files.any?
+      errors_and_warnings['PHP'] = check_php_files(php_files)
+    end
+
+    output_results(errors_and_warnings)
+  end
+end

data/lib/sumodev/version.rb

@@ -1,3 +1,3 @@
 module Sumodev
-  VERSION = "0.8.2"
+  VERSION = "0.9.0"
 end

data/lib/sumodev.rb

@@ -9,6 +9,7 @@ class Sumodev < Thor
     autoload :Push, 'sumodev/commands/push'
     autoload :Project, 'sumodev/commands/project'
     autoload :Box, 'sumodev/commands/box'
+    autoload :Hooks, 'sumodev/commands/hooks'
   end
 
   register Commands::Fork, 'fork', 'fork <command>', 'All commands concerning Fork applications'
@@ -17,5 +18,5 @@ class Sumodev < Thor
   register Commands::Push, 'push', 'push <server>', 'Push the SSH keys to the server'
   register Commands::Project, 'project', 'project <command>', 'All commands concerning projects'
   register Commands::Box, 'box', 'box <command>', 'All commands concerning our Vagrant install'
+  register Commands::Hooks, 'hooks', 'hooks <command>', 'All commands concerning our git hooks'
 end
-

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: sumodev
 version: !ruby/object:Gem::Version
-  version: 0.8.2
+  version: 0.9.0
 platform: ruby
 authors:
 - Jan De Poorter
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-01-05 00:00:00.000000000 Z
+date: 2015-03-26 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
@@ -79,6 +79,7 @@ files:
 - lib/sumodev/commands/box.rb
 - lib/sumodev/commands/factr.rb
 - lib/sumodev/commands/fork.rb
+- lib/sumodev/commands/hooks.rb
 - lib/sumodev/commands/project.rb
 - lib/sumodev/commands/push.rb
 - lib/sumodev/commands/ssh.rb