Lint/UselessAccessModifier

The module specifies 'module_function' at the beginning so it doesn't
need to be repeated. Also all of the functions are private when included
in a class.

Author: joshcooper

.rubocop_todo.yml

@@ -643,12 +643,6 @@ Lint/ToJSON:
 Lint/UnusedMethodArgument:
   Enabled: false
 
-# This cop supports safe auto-correction (--auto-correct).
-# Configuration parameters: ContextCreatingMethods, MethodCreatingMethods.
-Lint/UselessAccessModifier:
-  Exclude:
-    - 'lib/hiera_puppet.rb'
-
 # This cop supports unsafe auto-correction (--auto-correct-all).
 Lint/UselessMethodDefinition:
   Exclude:

lib/hiera_puppet.rb

@@ -49,9 +49,6 @@ def parse_args(args)
     return [key, default, override]
   end
 
-  private
-  module_function
-
   def hiera
     @hiera ||= Hiera.new(:config => hiera_config)
   end