check_names 0.3.0 → 0.4.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 4cb09b3fed90bd9fa021f047530c0caa584f03b077c73a048dfd60fe6e059a97
-  data.tar.gz: d90ea5f678f36bd1e6144fba95cfcb9a5136dcd7f63b3ef31a60fbe8e89ab74b
+  metadata.gz: 8e8b673b887e26b3ec7100c88369de69d82f985a65389b9bbf4d901e0dc76a53
+  data.tar.gz: c5c5d6f4157348de247b653598360cc6da83d56940fa85a0e4a0367b6b13a2c7
 SHA512:
-  metadata.gz: 8b4baf708e2ef476305f901103ec360809a4ca79c299a81ed6b4d241c19649d6e0600ee8eba41820f506a9329cf1d230ab1a11cd16845748f0615e74bdb25918
-  data.tar.gz: 730e74250b8f7c7ddb87fcc42fb683edc60e80fb1b25001e6c399b9705d95643c5798e20daa2170772b4cb3117b590b9a4e58f248c6ef400aeebf52b491a123e
+  metadata.gz: ed35fb48743e2651df7d8d03fccc9394ef69d69ce5268059d11542a84ec941759c03092120b1cedea4c339b4b1426446d5e857a74bcab46a5e8111fb3468f40d
+  data.tar.gz: db674a40f05f65f22705ed8eca975c6452a02bc2322102fe889990e7313d8cbb1d6974f8cba57498ebf875d20a89e0c9c43d99e3180c84aacf3f42989d7f079d

data/exe/check_names

@@ -22,33 +22,30 @@ Version #{CheckNames.version}
 Usage:
   check_names [OPTIONS]... [names]...
 
-check_names exits with a zero return code (success) if no conflicting gem
-name exists. Otherwise it exits with a non-zero return code, with one 
-of the following meanings:
+check_names looks to see whether the names provided are already in use.
+It outputs the usage (if any) of each name on $STDOUT. It exits with a 
+zero return code if none of the names are already in use. Otherwise it 
+exits with return code 1. 
 
-1  A gem exists on Rubygems with this name
-2  You have a github repository of this name
-3  You have an executable (reachable via \$PATH) with this name
-4  You have a shell function with this name
-5  You have an alias with this name
-6  This name is used by a Ruby standard class
+check_names checks for each of the following uses:
 
-If more than one name was given, the return code is the maximum return code
-received for any name.
+-  The name of a Ruby standard class
+-  The name of a gem on RubyGems
+-  The name of one of your Github repositories
+-  The name of a sehll alias
+-  The name of a shell function
+-  The name of an executable (reachable via \$PATH)
 
 The following options are available:
 
 --help  -h  Print this help information
---quiet -q  Suppress messages about the status of the name check
---all   -a  Normally, check_names stops at the first use it finds.
-            With --all, it checks and reports every use
+--all   -a  Report all uses of the name. (Otherwise, check_names stops at 
+            the first use it finds.)
   HELP
 end
 
 def main(args)
   args = args.map { |arg| arg.downcase }
-  quiet = args.index { |arg| %w{-q --quiet}.include?(arg) }
-  args.delete_at(quiet) if quiet
   all = args.index { |arg| %w{-a --all}.include?(arg) }
   args.delete_at(all) if all
   if args.size == 0 || %w{-h --help help}.include?(args.first)
@@ -56,27 +53,26 @@ def main(args)
     exit
   end
 
-  if all
-    results = GemName.check_names_all(*args)
-  else
-    results = GemName.check_names(*args)
-  end
-  
   rc = 0
-
-  types = []
-  results.each.with_index(1) do |result, count|
-    types << result[:description] if result[:result]
-    rc = result[:check] if result[:check] > rc
-    if count >= results.size || results[count][:name] != result[:name]
-      unless quiet
-        types << "Unused" if types.size == 0
-        puts "#{result[:name]}: #{types.join(', ')}"
+  args.each.with_index(0) do |arg, i|
+    results = []
+    print "#{arg}: "
+    checks_printed_count = 0
+    (1...(CheckNames.check_methods.size)).each do |check|
+      result = CheckNames.check_name(check, arg)
+      if result[:result]
+        print ", " if checks_printed_count > 0
+        print result[:description]
+        checks_printed_count += 1
+      end
+      if checks_printed_count > 0
+        rc = 1
+        break unless all
       end
-      types = []
     end
+    print CheckNames.check_method(0)[:description] unless checks_printed_count > 0
+    puts
   end
-  rc = 1 if rc > 0 && (all || args.size > 1)
   exit(rc)
 end
 

data/lib/check_names/version.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 module CheckNames
-  VERSION = "0.3.0"
+  VERSION = "0.4.0"
 
   def self.version
     VERSION

data/lib/check_names.rb

@@ -4,7 +4,7 @@ require_relative "check_names/version"
 require 'shellwords'
 require 'httparty'
 
-module GemName
+module CheckNames
   class Error < StandardError; end
 
   def self.ruby_gem_exists?(name)
@@ -48,24 +48,38 @@ module GemName
 
   def self.standard_class_exists?(name)
     class_name = name.split('_').map(&:capitalize).join
-    Module.const_defined?(class_name)
+    # Do it this way so the Class space isn't polluted with the classes of this script
+    command = "ruby -e \"puts Module.const_defined?(\\\"#{class_name}\\\").inspect\""
+    begin
+      eval(`#{command}`.chomp)
+    rescue
+      false
+    end
   end
 
   CHECK_METHODS = [
-    {method: nil, description: "Unused"}, # 0
-    {method: :ruby_gem_exists?, description: "RubyGem"}, # 1
-    {method: :github_repository_exists?, description: "Github repository"}, # 2
-    {method: :executable_exists?, description: "Executable"}, # 3
-    {method: :function_exists?, description: "Function"}, # 4
-    {method: :alias_exists?, description: "Alias"}, # 5
-    {method: :standard_class_exists?, description: "Class"} # 6
+    {method: nil, description: "Unused"},
+    {method: :standard_class_exists?, description: "Class"},
+    {method: :ruby_gem_exists?, description: "RubyGem"},
+    {method: :github_repository_exists?, description: "Github repository"},
+    {method: :alias_exists?, description: "Alias"},
+    {method: :function_exists?, description: "Function"},
+    {method: :executable_exists?, description: "Executable"}
   ]
 
+  def self.check_methods
+    CHECK_METHODS
+  end
+
+  def self.check_method(i)
+    CHECK_METHODS[i]
+  end
+
   def self.unused_result
     {check: 0, description: CHECK_METHODS[0][:description], result: true}
   end
 
-  def self.check_name_once(check, name)
+  def self.check_name(check, name)
     method = CHECK_METHODS[check] && CHECK_METHODS[check][:method]
     if method
       result = {check: check, description: CHECK_METHODS[check][:description], result: send(method, name)}
@@ -75,40 +89,4 @@ module GemName
     result[:name] = name
     result
   end
-
-  def self.check_name(name)
-    name = name.downcase
-    result = unused_result.dup
-    (1...(CHECK_METHODS.size)).each do |check|
-      res = check_name_once(check, name)
-      if res[:result]
-        result = res
-        break
-      end
-    end
-    result[:name] = name
-    result
-  end
-
-  def self.check_name_all(name)
-    (1...(CHECK_METHODS.size)).map do |check|
-      check_name_once(check, name)
-    end
-  end
-
-  def self.check_names(*names)
-    results = []
-    names.each do |name|
-      results << check_name(name)
-    end
-    results
-  end
-
-  def self.check_names_all(*names)
-    results = []
-    names.each do |name|
-      results += check_name_all(name)
-    end
-    results
-  end
 end

data/sig/check_names.rbs

@@ -1,4 +1,4 @@
-module GemName
+module CheckNames
   VERSION: String
   # See the writing guide of rbs: https://github.com/ruby/rbs#guides
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: check_names
 version: !ruby/object:Gem::Version
-  version: 0.3.0
+  version: 0.4.0
 platform: ruby
 authors:
 - Richard LeBer