Remove .nil? and add some yard explaining what this method is

Signed-off-by: Tim Smith <tsmith@chef.io>

1 files changed, 9 insertions, 1 deletions

diff --git a/chef-config/lib/chef-config/mixin/fuzzy_hostname_matcher.rb b/chef-config/lib/chef-config/mixin/fuzzy_hostname_matcher.rb
index 05188d44aa..533db85860 100644
--- a/chef-config/lib/chef-config/mixin/fuzzy_hostname_matcher.rb
+++ b/chef-config/lib/chef-config/mixin/fuzzy_hostname_matcher.rb
@@ -20,8 +20,16 @@ module ChefConfig
   module Mixin
     module FuzzyHostnameMatcher
 
+      #
+      # Check to see if a hostname matches a match string. Used to see if hosts fall under our no_proxy config
+      #
+      # @param [String] hostname the hostname to check
+      # @param [String] matches the pattern to match
+      #
+      # @return [Boolean]
+      #
       def fuzzy_hostname_match_any?(hostname, matches)
-        unless hostname.nil? || matches.nil?
+        if hostname && matches
           return matches.to_s.split(/\s*,\s*/).compact.any? do |m|
             fuzzy_hostname_match?(hostname, m)
           end