cop for gem fetched from a git source

code fix for aesthetic & conventions

remove unused proc

2 files changed, 29 insertions, 0 deletions

diff --git a/rubocop/cop/gem_fetcher.rb b/rubocop/cop/gem_fetcher.rb
new file mode 100644
index 00000000000..4a63c760744
--- /dev/null
+++ b/rubocop/cop/gem_fetcher.rb
@@ -0,0 +1,28 @@
+module RuboCop
+  module Cop
+    # Cop that checks for all gems specified in the Gemfile, and will
+    # alert if any gem is to be fetched not from the RubyGems index.
+    # This enforcement is done so as to minimize external build
+    # dependencies and build times.
+    class GemFetcher < RuboCop::Cop::Cop
+      MSG = 'Do not use gems from git repositories, only use gems from RubyGems.'
+
+      GIT_KEYS = [:git, :github]
+
+      def on_send(node)
+        file_path = node.location.expression.source_buffer.name
+        return unless file_path.end_with?("Gemfile")
+
+        func_name = node.children[1]
+        return unless func_name == :gem
+
+        node.children.last.each_node(:pair) do |pair|
+          key_name = pair.children[0].children[0].to_sym
+          if GIT_KEYS.include?(key_name)
+            add_offense(node, :selector)
+          end
+        end
+      end
+    end
+  end
+end
diff --git a/rubocop/rubocop.rb b/rubocop/rubocop.rb
index 7922e19768b..7f20754ee51 100644
--- a/rubocop/rubocop.rb
+++ b/rubocop/rubocop.rb
@@ -1,3 +1,4 @@
 require_relative 'migration_helpers'
 require_relative 'cop/migration/add_index'
 require_relative 'cop/migration/column_with_default'
+require_relative 'cop/gem_fetcher'