git-hook 0.1.8 → 0.1.9

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: ebab726ddd8101a28247bc96a20628711c56f22f
-  data.tar.gz: c29c99fa2fcb931101290d18f40c3a75c025f713
+  metadata.gz: 59be1b7b50d12a2c488354d0ae590fdf4d1f880b
+  data.tar.gz: 0f0788df757f05f8e2d1b8d73cbe82d60ee68198
 SHA512:
-  metadata.gz: ef86bf172e83076e1af209611564b93b4e30345c65d95cf39c003df27c52f632ffa9ab6df3205a9d097f4bdb57d43dc36e4eb69e3494065dd654ff0b5e5114df
-  data.tar.gz: 448237492eaa4dde373d7a2ffa93f4c38af944053f4cb728e1dbcfb773d53eda613cb44fe565a48ace7f1a2b73fedb0b1d676f37cdff7b0626834a1cd383ff59
+  metadata.gz: 3d685f15442defe685c1c5cddbe6378ae26de64236b2c772d526c997eb0d9da9978e65fc0750e9962ecb74147cb611af7ea47f310cada8476d9a6dd3af8a1162
+  data.tar.gz: 8963581e1246700d95933bc48c1e925d609408cc8e7d1e22023d102ba2d74fa1e9cdcd6b4c9e083fa1e6f2bb504ba61096642655f7feefbba4e1ccb9cf9e7ce3

data/README.md

@@ -45,7 +45,7 @@ Help:
       install  -- Init githook, create .githook folder, prepare template files
       setup    -- Setup hooks, copy hooks from .githook/hooks to .git/hooks
       backup   -- Backup old hooks in .git/hooks
-      clean  -- Clear backup hooks in .git/hooks
+      clean    -- Clear backup hooks in .git/hooks
       disable  -- Disable hooks: [HOOKS=pre_commit,commit_msg] githook disable
       enable   -- Enable hooks: [HOOKS=pre_commit,commit_msg] githook enable
       list     -- List all hooks
@@ -57,7 +57,11 @@ Getting started:
     $ githook install
     $ githook setup
 
-Demos:
+Modify `.githook/config.rb` according to your requirement:
+
+    $ vim .githook/config.rb
+
+## Demo
 
 1. install, setup hooks
 
@@ -241,6 +245,31 @@ Demo:
 
 1. [How to Build a Gem](note/how-to-build-a-gem.md)
 
+## Releases
+
+### 0.1.9
+
+1. Add a `changed_files` util method to get the added or modified files, to make rubocop only check changed files instead of all files.
+
+### 0.1.8
+
+1. Rename `githook clearup` command to `githook clean`
+
+1. Unify `githook pre_commit`, `githook prepare_commit_msg`, `githook commit_msg` commands to `githook run` but with different ENV value
+
+        githook pre_commit          -->  HOOK=pre_commit githook run
+        githook prepare_commit_msg  -->  HOOK=prepare_commit_msg githook run
+        githook commit_msg          -->  HOOK=commit_msg githook run
+
+1. Support gitlab type branch name, for example "100-support-gitlab-branch", it will generate "FEATURE #100 - Support gitlab branch" commit message by `prepare_commit_msg` hook, uncomment `"prepare_commit_msg:prepare_for_gitlab_branch"` line and comment `"prepare_commit_msg:prepare_for_ekohe_branch"` line in `.githook/config.rb` to make it work
+
+        // .githook/config
+        set :prepare_commit_msg, fetch(:prepare_commit_msg, []).push(
+          # comment following lines if you want to skip it
+          # "prepare_commit_msg:prepare_for_ekohe_branch"
+          "prepare_commit_msg:prepare_for_gitlab_branch"
+        )
+
 ## License
 
 The gem is available as open source under the terms of the [MIT License](http://opensource.org/licenses/MIT).

data/Rakefile

@@ -1,6 +1,6 @@
-require "bundler/gem_tasks"
-require "rspec/core/rake_task"
+require 'bundler/gem_tasks'
+require 'rspec/core/rake_task'
 
 RSpec::Core::RakeTask.new(:spec)
 
-task :default => :spec
+task default: :spec

data/bin/githook

@@ -1,8 +1,8 @@
 #!/usr/bin/env ruby
 
-require "githook"
+require 'githook'
 
-task_name = ARGV[0] || "help"
+task_name = ARGV[0] || 'help'
 if Rake::Task.task_defined?(task_name)
   Rake::Task[task_name].invoke
 else

data/githook.gemspec

@@ -1,18 +1,18 @@
-# coding: utf-8
-lib = File.expand_path('../lib', __FILE__)
+
+lib = File.expand_path('lib', __dir__)
 $LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
 require 'githook/version'
 
 Gem::Specification.new do |spec|
-  spec.name          = "git-hook"
+  spec.name          = 'git-hook'
   spec.version       = Githook::VERSION
-  spec.authors       = ["baurine"]
-  spec.email         = ["2008.hbl@gmail.com"]
+  spec.authors       = ['baurine']
+  spec.email         = ['2008.hbl@gmail.com']
 
-  spec.summary       = %q{Setup git hooks easily}
-  spec.description   = %q{Help to setup git hooks easily, base on Rake, inspired from Capistrano.}
-  spec.homepage      = "https://github.com/baurine/githook"
-  spec.license       = "MIT"
+  spec.summary       = 'Setup git hooks easily'
+  spec.description   = 'Help to setup git hooks easily, base on Rake, inspired from Capistrano.'
+  spec.homepage      = 'https://github.com/baurine/githook'
+  spec.license       = 'MIT'
 
   # Prevent pushing this gem to RubyGems.org. To allow pushes either set the 'allowed_push_host'
   # to allow pushing to a single host or delete this section to allow pushing to any host.
@@ -28,9 +28,9 @@ Gem::Specification.new do |spec|
   end
   spec.executables   = spec.files.grep(%r{^bin/}) { |f| File.basename(f) }
   spec.test_files    = spec.files.grep(%r{^(test|spec|features)/})
-  spec.require_paths = ["lib"]
+  spec.require_paths = ['lib']
 
-  spec.add_development_dependency "bundler", "~> 1.14"
-  spec.add_development_dependency "rake", "~> 10.0"
-  spec.add_development_dependency "rspec", "~> 3.0"
+  spec.add_development_dependency 'bundler', '~> 1.14'
+  spec.add_development_dependency 'rake', '~> 10.0'
+  spec.add_development_dependency 'rspec', '~> 3.0'
 end

data/lib/githook.rb

@@ -1,9 +1,9 @@
-require "rake"
-require "githook/version"
-require "githook/context"
-require "githook/util"
-require "githook/tasks"
-require "githook/config"
+require 'rake'
+require 'githook/version'
+require 'githook/context'
+require 'githook/util'
+require 'githook/tasks'
+require 'githook/config'
 
 module Githook
   # Your code goes here...

data/lib/githook/config.rb

@@ -1,10 +1,10 @@
 # we must be very careful to load outside ruby code
 # because they are out of our control
 # only load outside "*.rake" when there are ".git" and ".githook" folder, and target task isn't "install"
-if Dir.exist?(".git") && Dir.exist?(".githook") && ARGV[0] != "install"
+if Dir.exist?('.git') && Dir.exist?('.githook') && ARGV[0] != 'install'
   begin
-    load ".githook/config.rb"
-  rescue => e
+    load '.githook/config.rb'
+  rescue StandardError => e
     puts "Error: #{e.message} in .githook/config.rb"
     exit 1
   end

data/lib/githook/context.rb

@@ -4,11 +4,11 @@ module Githook
       @env ||= {}
       @env[key] = value
     end
-  
+
     def self.fetch(key, def_val)
       @env ||= {}
       @env[key] || def_val
-    end 
+    end
   end
 end
 

data/lib/githook/tasks.rb

@@ -4,18 +4,18 @@
 Rake::TaskManager.record_task_metadata = true
 
 # load rake files from lib
-rake_files_pattern = File.dirname(__FILE__) + "/tasks/*.rake"
+rake_files_pattern = File.dirname(__FILE__) + '/tasks/*.rake'
 # => gems/git-hook-0.1.1/lib/githook/tasks/*.rake
 Dir.glob(rake_files_pattern).each { |r| load r }
 
 # we must be very careful to load outside ruby code
 # because they are out of our control
 # only load outside "*.rake" when there are ".git" and ".githook" folder, and target task isn't "install"
-if Dir.exist?(".git") && Dir.exist?(".githook") && ARGV[0] != "install"
-  Dir.glob(".githook/tasks/**/*.rake").each do |rake|
+if Dir.exist?('.git') && Dir.exist?('.githook') && ARGV[0] != 'install'
+  Dir.glob('.githook/tasks/**/*.rake').each do |rake|
     begin
       load rake
-    rescue => e
+    rescue StandardError => e
       puts "Error: #{e.message} in #{rake}"
       exit 1
     end

data/lib/githook/tasks/1-pre-commit.rake

@@ -1,19 +1,19 @@
 namespace :pre_commit do
-  desc "Check ruby code style by rubocop"
+  desc 'Check ruby code style by rubocop'
   task :rubocop do |t|
     Githook::Util.log_task(t.name)
-    exit 1 unless system("bundle exec rubocop")
+    exit 1 unless system("bundle exec rubocop #{Githook::Util.changed_ruby_files}")
   end
 
-  desc "Test ruby code by rspec"
+  desc 'Test ruby code by rspec'
   task :rspec do |t|
     Githook::Util.log_task(t.name)
-    exit 1 unless system("bundle exec rspec")
+    exit 1 unless system('bundle exec rspec')
   end
 
-  desc "Check java code style by checkstyle"
+  desc 'Check java code style by checkstyle'
   task :checkstyle do |t|
     Githook::Util.log_task(t.name)
-    exit 1 unless system("./gradlew checkstyle")
+    exit 1 unless system('./gradlew checkstyle')
   end
 end

data/lib/githook/tasks/2-prepare-commit-msg.rake

@@ -13,9 +13,9 @@ namespace :prepare_commit_msg do
       issue_num = match_group[2]
       issue_content = match_group[3]
 
-      issue_type = "BUG" if issue_type == "HOTFIX"
+      issue_type = 'BUG' if issue_type == 'HOTFIX'
       issue_num = " \##{issue_num}" unless issue_num.empty?
-      issue_content = issue_content.tr("_", " ").strip.capitalize
+      issue_content = issue_content.tr('_', ' ').strip.capitalize
 
       "#{issue_type}#{issue_num} - #{issue_content}"
     else
@@ -34,9 +34,9 @@ namespace :prepare_commit_msg do
       issue_num = match_group[1]
       issue_content = match_group[2]
 
-      issue_type = "FEATURE"
+      issue_type = 'FEATURE'
       issue_num = "\##{issue_num}"
-      issue_content = issue_content.tr("-", " ").strip.capitalize
+      issue_content = issue_content.tr('-', ' ').strip.capitalize
 
       "#{issue_type} #{issue_num} - #{issue_content}"
     else
@@ -45,7 +45,7 @@ namespace :prepare_commit_msg do
     end
   end
 
-  desc "Prepare commit msg for ekohe type branch"
+  desc 'Prepare commit msg for ekohe type branch'
   task :prepare_for_ekohe_branch do |t|
     Githook::Util.log_task(t.name)
 
@@ -54,13 +54,13 @@ namespace :prepare_commit_msg do
     if Githook::Util.commit_msg_empty?(commit_msg)
       branch_name = Githook::Util.branch_name
       pre_msg = gen_pre_msg_for_ekohe_branch(branch_name)
-      puts "pre-msg:"
+      puts 'pre-msg:'
       puts pre_msg
       Githook::Util.prefill_msg(commit_msg_file, pre_msg)
     end
   end
 
-  desc "Prepare commit msg for gitlab type branch"
+  desc 'Prepare commit msg for gitlab type branch'
   task :prepare_for_gitlab_branch do |t|
     Githook::Util.log_task(t.name)
 
@@ -69,7 +69,7 @@ namespace :prepare_commit_msg do
     if Githook::Util.commit_msg_empty?(commit_msg)
       branch_name = Githook::Util.branch_name
       pre_msg = gen_pre_msg_for_gitlab_branch(branch_name)
-      puts "pre-msg:"
+      puts 'pre-msg:'
       puts pre_msg
       Githook::Util.prefill_msg(commit_msg_file, pre_msg)
     end

data/lib/githook/tasks/3-commit-msg.rake

@@ -1,15 +1,15 @@
 namespace :commit_msg do
   def check_msg_for_ekohe_format?(commit_msg_arr)
     def_ekohe_msg_summary_reg = /^(FEATURE|BUG|MISC|REFACTOR)(\s#\d+)* - ([A-Z].*)[^.]$/
-    def_ekohe_msg_format = "FEAUTER|BUG|MISC|REFACTOR #issue_num - Summary"
+    def_ekohe_msg_format = 'FEAUTER|BUG|MISC|REFACTOR #issue_num - Summary'
     def_ekohe_body_reg = /^- ([a-z].*)[^.]$/
-    def_ekohe_body_format = "- detail"
+    def_ekohe_body_format = '- detail'
 
-    summary = commit_msg_arr[0] || ""
-    second_line = commit_msg_arr[1] || ""
+    summary = commit_msg_arr[0] || ''
+    second_line = commit_msg_arr[1] || ''
     body = commit_msg_arr[2..-1] || []
 
-    valid = summary.start_with?("Merge branch") || def_ekohe_msg_summary_reg.match(summary)
+    valid = summary.start_with?('Merge branch') || def_ekohe_msg_summary_reg.match(summary)
     unless valid
       puts "Commit message summary \"#{summary}\" format isn't correct."
       puts "Expected format: \"#{def_ekohe_msg_format}\""
@@ -18,27 +18,26 @@ namespace :commit_msg do
 
     valid = second_line.strip.empty?
     unless valid
-      puts "Commit message the first line after summary should be blank."
+      puts 'Commit message the first line after summary should be blank.'
       return false
     end
 
     body.each do |line|
-      unless def_ekohe_body_reg.match(line)
-        puts "Commit message body line \"#{line}\" format isn't correct."
-        puts "Expected format: \"#{def_ekohe_body_format}\""
-        return false
-      end
+      next if def_ekohe_body_reg.match(line)
+      puts "Commit message body line \"#{line}\" format isn't correct."
+      puts "Expected format: \"#{def_ekohe_body_format}\""
+      return false
     end
     true
   end
 
-  desc "Check commit msg style for ekohe format"
+  desc 'Check commit msg style for ekohe format'
   task :check_msg_for_ekohe_format do |t|
     Githook::Util.log_task(t.name)
 
     commit_msg_file = Githook::Util.commit_msg_file
     commit_msg = Githook::Util.get_commit_msg(commit_msg_file)
-    puts "commit-msg:"
+    puts 'commit-msg:'
     puts commit_msg.join("\n")
 
     # can't use return in block

data/lib/githook/tasks/install.rake

@@ -1,23 +1,23 @@
-desc "Init githook, create .githook folder, prepare template files"
+desc 'Init githook, create .githook folder, prepare template files'
 task :install do
   # step 1, check whether Dir.pwd is in git repo root folder
-  git_path = ".git"
-  unless Dir.exists?(git_path)
+  git_path = '.git'
+  unless Dir.exist?(git_path)
     puts "It isn't in a git repo root folder."
     exit 1
   end
 
   # step 2, check whether ".githook" folder already exists
-  githook_path = ".githook"
-  if Dir.exists?(githook_path)
-    print ".githook already exists, do you want to override it? [y/(n)]: "
+  githook_path = '.githook'
+  if Dir.exist?(githook_path)
+    print '.githook already exists, do you want to override it? [y/(n)]: '
     choice = STDIN.gets
-    exit 0 if ["n", "N", "\n"].include?(choice[0])
+    exit 0 if %W[n N \n].include?(choice[0])
     FileUtils.rm_r(githook_path)
   end
 
   # setup 3, copy templates to .githook
-  templates_path = File.expand_path("../../templates", __FILE__)
+  templates_path = File.expand_path('../templates', __dir__)
   FileUtils.cp_r(templates_path, githook_path)
-  puts "Create .githook folder."
+  puts 'Create .githook folder.'
 end

data/lib/githook/tasks/run.rake

@@ -1,6 +1,6 @@
-desc "Run hook tasks"
-task :run do |t|
-  hook = ENV["HOOK"]
+desc 'Run hook tasks'
+task :run do |_t|
+  hook = ENV['HOOK']
   if hook
     Githook::Util.log_task(hook)
     Githook::Util.run_tasks(hook.to_sym)

data/lib/githook/tasks/setup.rake

@@ -1,16 +1,16 @@
-desc "Check whether .githook/hooks folder exists"
+desc 'Check whether .githook/hooks folder exists'
 task :check_githook_folder do
-  hooks_path = ".githook/hooks"
-  unless Dir.exists?(hooks_path)
+  hooks_path = '.githook/hooks'
+  unless Dir.exist?(hooks_path)
     puts "There isn't a .githook/hooks folder."
     exit 1
   end
 end
 
-desc "Check whether .git/hooks folder exists"
+desc 'Check whether .git/hooks folder exists'
 task :check_git_folder do
-  git_path = ".git/hooks"
-  unless Dir.exists?(git_path)
+  git_path = '.git/hooks'
+  unless Dir.exist?(git_path)
     puts "There isn't a .git/hooks folder."
     exit 1
   end
@@ -18,27 +18,27 @@ end
 
 #################################################################
 
-desc "Setup hooks, copy hooks from .githook/hooks to .git/hooks"
-task :setup => [:check_githook_folder, :check_git_folder] do
+desc 'Setup hooks, copy hooks from .githook/hooks to .git/hooks'
+task setup: %i[check_githook_folder check_git_folder] do
   # setup 1, check whether has ".githook/hooks" and ".git" folder
   # => [:check_githook_folder, :check_git_folder]
 
   # setup 2, backup hooks
-  puts "Backup old hooks:"
+  puts 'Backup old hooks:'
   Rake::Task[:backup].invoke
 
   # setup 3, copy hooks to .git/hooks
-  FileUtils.cp_r(".githook/hooks", ".git")
+  FileUtils.cp_r('.githook/hooks', '.git')
 end
 
-desc "Backup old hooks in .git/hooks"
-task :backup => :check_git_folder do
+desc 'Backup old hooks in .git/hooks'
+task backup: :check_git_folder do
   has_backup = false
-  Dir.glob(".git/hooks/*").each do |path|
-    file_name = path.split("/").last
-    next if file_name.include?(".")
+  Dir.glob('.git/hooks/*').each do |path|
+    file_name = path.split('/').last
+    next if file_name.include?('.')
 
-    appendix = ".#{Time.now.strftime("%Y%m%d%H%m%S")}.bak"
+    appendix = ".#{Time.now.strftime('%Y%m%d%H%m%S')}.bak"
     puts "Backup old #{file_name} to #{file_name}#{appendix}"
     FileUtils.cp(path, "#{path}#{appendix}")
     has_backup = true
@@ -47,10 +47,10 @@ task :backup => :check_git_folder do
   puts "You can run 'githook clean' to delete these backup." if has_backup
 end
 
-desc "Clear backup hooks in .git/hooks"
-task :clean => :check_git_folder do
-  backup = Dir.glob(".git/hooks/*.bak")
-  Githook::Util.interactive_delete_files(backup, "backup hooks")
+desc 'Clear backup hooks in .git/hooks'
+task clean: :check_git_folder do
+  backup = Dir.glob('.git/hooks/*.bak')
+  Githook::Util.interactive_delete_files(backup, 'backup hooks')
 end
 
 # all hooks
@@ -73,14 +73,14 @@ end
 #   post_update
 # )
 
-desc "Disable hooks: [HOOKS=pre_commit,commit_msg] githook disable"
-task :disable => :check_git_folder do
-  target_hooks = (ENV["HOOKS"] || "").split(",")
+desc 'Disable hooks: [HOOKS=pre_commit,commit_msg] githook disable'
+task disable: :check_git_folder do
+  target_hooks = (ENV['HOOKS'] || '').split(',')
   target_hooks = Githook::Util.all_hooks if target_hooks.empty?
 
   target_hooks.each do |hook|
-    hook_path = File.join(".git/hooks", hook.gsub("_", "-"))
-    disable_path = hook_path + ".disable"
+    hook_path = File.join('.git/hooks', hook.tr('_', '-'))
+    disable_path = hook_path + '.disable'
     if File.file?(hook_path)
       FileUtils.mv(hook_path, disable_path)
       puts "Disable #{hook} hook."
@@ -92,14 +92,14 @@ task :disable => :check_git_folder do
   end
 end
 
-desc "Enable hooks: [HOOKS=pre_commit,commit_msg] githook enable"
-task :enable => :check_git_folder do
-  target_hooks = (ENV["HOOKS"] || "").split(",")
+desc 'Enable hooks: [HOOKS=pre_commit,commit_msg] githook enable'
+task enable: :check_git_folder do
+  target_hooks = (ENV['HOOKS'] || '').split(',')
   target_hooks = Githook::Util.all_hooks if target_hooks.empty?
 
   target_hooks.each do |hook|
-    hook_path = File.join(".git/hooks", hook.gsub("_", "-"))
-    disable_path = hook_path + ".disable"
+    hook_path = File.join('.git/hooks', hook.tr('_', '-'))
+    disable_path = hook_path + '.disable'
     if File.file?(hook_path)
       puts "#{hook} hook is arleady enabled, skip."
     elsif File.file?(disable_path)
@@ -111,32 +111,32 @@ task :enable => :check_git_folder do
   end
 end
 
-desc "List all hooks"
-task :list => :check_git_folder do
+desc 'List all hooks'
+task list: :check_git_folder do
   enabled_hooks = []
   disabled_hooks = []
   all_hooks = Githook::Util.all_hooks
   all_hooks.each do |hook|
-    hook_path = File.join(".git/hooks", hook.gsub("_", "-"))
-    disable_path = hook_path + ".disable"
+    hook_path = File.join('.git/hooks', hook.tr('_', '-'))
+    disable_path = hook_path + '.disable'
     if File.file?(hook_path)
       enabled_hooks << hook
     elsif File.file?(disable_path)
       disabled_hooks << hook
     end
   end
-  puts "Enabled hooks:"
+  puts 'Enabled hooks:'
   enabled_hooks.each { |h| puts "  * #{h}" }
-  puts "Disabled hooks:"
+  puts 'Disabled hooks:'
   disabled_hooks.each { |h| puts "  * #{h}" }
 end
 
-desc "Version"
+desc 'Version'
 task :version do
   puts Githook::VERSION
 end
 
-TASKS_NAME = %w(
+TASKS_NAME = %w[
   install
   setup
   backup
@@ -146,12 +146,12 @@ TASKS_NAME = %w(
   list
   version
   help
-)
-desc "Help"
+].freeze
+desc 'Help'
 task :help do
-  puts "Usage: githook task_name"
+  puts 'Usage: githook task_name'
   puts
-  puts "task_name:"
+  puts 'task_name:'
   left_len = TASKS_NAME.map(&:length).max + 2
   TASKS_NAME.each do |task_name|
     task = Rake::Task[task_name]

data/lib/githook/templates/config.rb

@@ -1,19 +1,20 @@
 set :pre_commit, fetch(:pre_commit, []).push(
   # uncomment following lines if it is a ruby project
-  # "pre_commit:rubocop",
-  # "pre_commit:rspec",
+  # 'pre_commit:rubocop',
+  # 'pre_commit:rspec',
 
   # uncomment following lines if it is a java project built by gradle
-  # "pre_commit:checkstyle"
+  # 'pre_commit:checkstyle'
 
-  # "pre_commit:check_branch_name"
+  # 'pre_commit:check_branch_name'
 )
+
 set :prepare_commit_msg, fetch(:prepare_commit_msg, []).push(
   # comment following lines if you want to skip it
-  "prepare_commit_msg:prepare_for_ekohe_branch"
+  'prepare_commit_msg:prepare_for_ekohe_branch'
   # "prepare_commit_msg:prepare_for_gitlab_branch"
 )
 set :commit_msg, fetch(:commit_msg, []).push(
   # comment following lines if you want to skip it
-  "commit_msg:check_msg_for_ekohe_format"
+  'commit_msg:check_msg_for_ekohe_format'
 )

data/lib/githook/util.rb

@@ -1,7 +1,7 @@
 module Githook
   class Util
     def self.log_task(task_name)
-      puts "[#{Time.now.strftime('%H:%m:%S')}] #{task_name.gsub('_', ' ')}"
+      puts "[#{Time.now.strftime('%H:%m:%S')}] #{task_name.tr('_', ' ')}"
     end
 
     def self.run_tasks(hook_stage)
@@ -18,15 +18,15 @@ module Githook
     #######################################################
 
     def self.interactive_delete_files(path_arr, type)
-      if path_arr.length == 0
+      if path_arr.empty?
         puts "There are no #{type}."
       else
         puts "There are following #{type}:"
         puts path_arr
-        print "Are you sure want to delete all of them [y/(n)]: "
+        print 'Are you sure want to delete all of them [y/(n)]: '
         # https://stackoverflow.com/a/40643667/2998877
         choice = STDIN.gets
-        return if ["N", "n", "\n"].include?(choice[0])
+        return if %W[n N \n].include?(choice[0])
 
         path_arr.each do |path|
           FileUtils.rm(path)
@@ -46,18 +46,18 @@ module Githook
 
     # include enabled_hooks and disabled_hooks
     def self.all_hooks
-      Dir.glob(".git/hooks/*")
-         .map { |path| path.split("/").last }
-         .select { |name| !name.include?(".") || name.include?(".disable") }
-         .map { |name| name.gsub(".disable", "") }
+      Dir.glob('.git/hooks/*')
+         .map { |path| path.split('/').last }
+         .select { |name| !name.include?('.') || name.include?('.disable') }
+         .map { |name| name.gsub('.disable', '') }
          .uniq
-         .map { |name| name.gsub("-", "_") }
+         .map { |name| name.tr('-', '_') }
     end
 
     #######################################################
 
     def self.commit_msg_file
-      ".git/COMMIT_EDITMSG"
+      '.git/COMMIT_EDITMSG'
     end
 
     def self.branch_name
@@ -67,9 +67,9 @@ module Githook
     def self.get_commit_msg(commit_msg_file)
       commit_msg = []
       # trim begining empty lines
-      File.open(commit_msg_file, "r") do |f|
+      File.open(commit_msg_file, 'r') do |f|
         f.readlines.each do |line|
-          next if line[0] == "#"
+          next if line[0] == '#'
           next if commit_msg.empty? && line.strip.empty?
           commit_msg << line
         end
@@ -96,12 +96,32 @@ module Githook
 
     # write the pre msg at the begining of commit_msg_file
     def self.prefill_msg(commit_msg_file, pre_msg)
-      File.open(commit_msg_file, "r+") do |f|
+      File.open(commit_msg_file, 'r+') do |f|
         ori_content = f.read
         f.seek(0, IO::SEEK_SET)
         f.puts pre_msg
         f.puts ori_content
       end
     end
+
+    # get changed files, include added or modified
+    def self.changed_files
+      added_or_modified_reg = /A|AM|^M/
+      `git status --porcelain`.split(/\n/)
+                              .select do |file_name_with_status|
+        file_name_with_status =~ added_or_modified_reg
+      end
+                              .map do |file_name_with_status|
+        file_name_with_status.split(' ')[1]
+      end
+    end
+
+    def self.changed_ruby_files
+      changed_files
+        .select do |file_name|
+        File.extname(file_name) == '.rb'
+      end
+        .join(' ')
+    end
   end
 end

data/lib/githook/version.rb

@@ -1,3 +1,3 @@
 module Githook
-  VERSION = "0.1.8"
+  VERSION = '0.1.9'.freeze
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: git-hook
 version: !ruby/object:Gem::Version
-  version: 0.1.8
+  version: 0.1.9
 platform: ruby
 authors:
 - baurine
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2018-09-02 00:00:00.000000000 Z
+date: 2019-03-29 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler