(PUP-11841) Add frozen_string_literal

Author: joshcooper

.rubocop_todo.yml

@@ -1217,12 +1217,6 @@ Style/FormatString:
 Style/FormatStringToken:
   Enabled: false
 
-# This cop supports safe auto-correction (--auto-correct).
-# Configuration parameters: EnforcedStyle.
-# SupportedStyles: always, always_true, never
-Style/FrozenStringLiteralComment:
-  Enabled: false
-
 # This cop supports safe auto-correction (--auto-correct).
 Style/GlobalStdStream:
   Exclude:

lib/hiera/puppet_function.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'hiera_puppet'
 
 # Provides the base class for the puppet functions hiera, hiera_array, hiera_hash, and hiera_include.

lib/hiera/scope.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'forwardable'
 class Hiera
   class Scope

lib/hiera_puppet.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 Puppet.features.hiera?
 require 'hiera/scope'
 require_relative 'puppet'

lib/puppet.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require_relative 'puppet/version'
 require_relative 'puppet/concurrent/synchronized'
 

lib/puppet/agent.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require_relative '../puppet/application'
 require_relative '../puppet/error'
 require_relative '../puppet/util/at_fork'

lib/puppet/agent/disabler.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require_relative '../../puppet/util/json_lockfile'
 
 # This module is responsible for encapsulating the logic for

lib/puppet/agent/locker.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require_relative '../../puppet/util/pidlock'
 require_relative '../../puppet/error'
 

lib/puppet/application.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'optparse'
 require_relative '../puppet/util/command_line'
 require_relative '../puppet/util/constant_inflector'

lib/puppet/application/agent.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require_relative '../../puppet/application'
 require_relative '../../puppet/daemon'
 require_relative '../../puppet/util/pidlock'

lib/puppet/application/apply.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require_relative '../../puppet/application'
 require_relative '../../puppet/configurer'
 require_relative '../../puppet/util/profiler/aggregate'

lib/puppet/application/catalog.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require_relative '../../puppet/application/indirection_base'
 
 class Puppet::Application::Catalog < Puppet::Application::IndirectionBase

lib/puppet/application/config.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require_relative '../../puppet/application/face_base'
 
 class Puppet::Application::Config < Puppet::Application::FaceBase

lib/puppet/application/describe.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require_relative '../../puppet/application'
 
 class Formatter

lib/puppet/application/device.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require_relative '../../puppet/application'
 require_relative '../../puppet/configurer'
 require_relative '../../puppet/util/network_device'

lib/puppet/application/doc.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require_relative '../../puppet/application'
 
 class Puppet::Application::Doc < Puppet::Application

lib/puppet/application/epp.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require_relative '../../puppet/application/face_base'
 require_relative '../../puppet/face'
 

lib/puppet/application/face_base.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require_relative '../../puppet/application'
 require_relative '../../puppet/face'
 require 'optparse'

lib/puppet/application/facts.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require_relative '../../puppet/application/indirection_base'
 
 class Puppet::Application::Facts < Puppet::Application::IndirectionBase

lib/puppet/application/filebucket.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require_relative '../../puppet/application'
 
 class Puppet::Application::Filebucket < Puppet::Application

lib/puppet/application/generate.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require_relative '../../puppet/application/face_base'
 
 # The Generate application. 

lib/puppet/application/help.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require_relative '../../puppet/application/face_base'
 
 class Puppet::Application::Help < Puppet::Application::FaceBase

lib/puppet/application/indirection_base.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require_relative '../../puppet/application/face_base'
 
 class Puppet::Application::IndirectionBase < Puppet::Application::FaceBase

lib/puppet/application/lookup.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require_relative '../../puppet/application'
 require_relative '../../puppet/pops'
 require_relative '../../puppet/node'

lib/puppet/application/module.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require_relative '../../puppet/application/face_base'
 
 class Puppet::Application::Module < Puppet::Application::FaceBase

lib/puppet/application/node.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require_relative '../../puppet/application/indirection_base'
 
 class Puppet::Application::Node < Puppet::Application::IndirectionBase

lib/puppet/application/parser.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require_relative '../../puppet/application/face_base'
 require_relative '../../puppet/face'
 

lib/puppet/application/plugin.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require_relative '../../puppet/application/face_base'
 class Puppet::Application::Plugin < Puppet::Application::FaceBase
   environment_mode :not_required

lib/puppet/application/report.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require_relative '../../puppet/application/indirection_base'
 
 class Puppet::Application::Report < Puppet::Application::IndirectionBase

lib/puppet/application/resource.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require_relative '../../puppet/application'
 
 class Puppet::Application::Resource < Puppet::Application

lib/puppet/application/script.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require_relative '../../puppet/application'
 require_relative '../../puppet/configurer'
 require_relative '../../puppet/util/profiler/aggregate'

lib/puppet/application/ssl.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require_relative '../../puppet/application'
 require_relative '../../puppet/ssl/oids'
 

lib/puppet/application_support.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'yaml'
 
 require_relative '../puppet'

lib/puppet/coercion.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 # Various methods used to coerce values into a canonical form.
 #
 # @api private

lib/puppet/compilable_resource_type.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require_relative '../puppet'
 # The CompilableResourceType module should be either included in a class or used as a class extension
 # to mark that the instance used as the 'resource type' of a resource instance

lib/puppet/concurrent.rb

@@ -1,2 +1,3 @@
+# frozen_string_literal: true
 module Puppet::Concurrent
 end

lib/puppet/concurrent/lock.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require_relative '../../puppet/concurrent/synchronized'
 
 module Puppet

lib/puppet/concurrent/synchronized.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 module Puppet
 module Concurrent
 

lib/puppet/configurer.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 # The client for interacting with the puppetmaster config server.
 require 'timeout'
 require_relative '../puppet/util'

lib/puppet/configurer/downloader.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require_relative '../../puppet/configurer'
 require_relative '../../puppet/resource/catalog'
 

lib/puppet/configurer/fact_handler.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require_relative '../../puppet/indirector/facts/facter'
 
 require_relative '../../puppet/configurer'

lib/puppet/configurer/plugin_handler.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 # Break out the code related to plugins.  This module is
 # just included into the agent, but having it here makes it
 # easier to test.

lib/puppet/confine.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 # The class that handles testing whether our providers
 # actually work or not.
 require_relative '../puppet/util'

lib/puppet/confine/any.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 class Puppet::Confine::Any < Puppet::Confine
   def self.summarize(confines)
     confines.inject(0) { |count, confine| count + confine.summary }

lib/puppet/confine/boolean.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require_relative '../../puppet/confine'
 
 # Common module for the Boolean confines. It currently

lib/puppet/confine/exists.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require_relative '../../puppet/confine'
 
 class Puppet::Confine::Exists < Puppet::Confine

lib/puppet/confine/false.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require_relative '../../puppet/confine/boolean'
 
 class Puppet::Confine::False < Puppet::Confine

lib/puppet/confine/feature.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require_relative '../../puppet/confine'
 
 class Puppet::Confine::Feature < Puppet::Confine

lib/puppet/confine/true.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require_relative '../../puppet/confine/boolean'
 
 class Puppet::Confine::True < Puppet::Confine

lib/puppet/confine/variable.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require_relative '../../puppet/confine'
 
 # Require a specific value for a variable, either a Puppet setting

lib/puppet/confine_collection.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 # Manage a collection of confines, returning a boolean or
 # helpful information.
 require_relative '../puppet/confine'

lib/puppet/confiner.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require_relative '../puppet/confine_collection'
 
 # The Confiner module contains methods for managing a Provider's confinement (suitability under given

lib/puppet/context.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require_relative '../puppet/thread_local'
 
 # Puppet::Context is a system for tracking services and contextual information

lib/puppet/context/trusted_information.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require_relative '../../puppet/trusted_external'
 
 # @api private

lib/puppet/daemon.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require_relative '../puppet/application'
 require_relative '../puppet/scheduler'
 

lib/puppet/data_binding.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require_relative '../puppet/indirector'
 
 # A class for managing data lookups

lib/puppet/datatypes.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 # Data types in the Puppet Language can have implementations written in Ruby
 # and distributed in puppet modules. A data type can be declared together with
 # its implementation by creating a file in 'lib/puppet/functions/<modulename>'.

lib/puppet/datatypes/error.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 Puppet::DataTypes.create_type('Error') do
   interface <<-PUPPET
     type_parameters => {

lib/puppet/datatypes/impl/error.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 class Puppet::DataTypes::Error
   attr_reader :msg, :kind, :issue_code, :details
   alias message msg

lib/puppet/defaults.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require_relative '../puppet/util/platform'
 
 module Puppet

lib/puppet/environments.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require_relative '../puppet/concurrent/synchronized'
 
 # @api private

lib/puppet/error.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 module Puppet
   # The base class for all Puppet errors. It can wrap another exception
   class Error < RuntimeError

lib/puppet/etc.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require_relative '../puppet/util/character_encoding'
 # Wrapper around Ruby Etc module allowing us to manage encoding in a single
 # place.

lib/puppet/external/dot.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 # rdot.rb
 #
 #

lib/puppet/face.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 # The public name of this feature is 'face', but we have hidden all the
 # plumbing over in the 'interfaces' namespace to make clear the distinction
 # between the two.

lib/puppet/face/catalog.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require_relative '../../puppet/indirector/face'
 
 Puppet::Indirector::Face.define(:catalog, '0.0.1') do

lib/puppet/face/catalog/select.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 # Select and show a list of resources of a given type.
 Puppet::Face.define(:catalog, '0.0.1') do
   action :select do

lib/puppet/face/config.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require_relative '../../puppet/face'
 require_relative '../../puppet/settings/ini_file'
 

lib/puppet/face/epp.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require_relative '../../puppet/face'
 require_relative '../../puppet/pops'
 require_relative '../../puppet/parser/files'

lib/puppet/face/facts.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require_relative '../../puppet/indirector/face'
 require_relative '../../puppet/node/facts'
 

lib/puppet/face/generate.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require_relative '../../puppet/face'
 require_relative '../../puppet/generate/type'
 

lib/puppet/face/help.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require_relative '../../puppet/face'
 require_relative '../../puppet/application/face_base'
 require_relative '../../puppet/util/constant_inflector'

lib/puppet/face/module.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require_relative '../../puppet/face'
 require_relative '../../puppet/module_tool'
 require_relative '../../puppet/util/colors'

lib/puppet/face/module/changes.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 Puppet::Face.define(:module, '1.0.0') do
   action(:changes) do
     summary _("Show modified files of an installed module.")

lib/puppet/face/module/install.rb

@@ -1,4 +1,5 @@
 # encoding: UTF-8
+# frozen_string_literal: true
 require_relative '../../../puppet/forge'
 require_relative '../../../puppet/module_tool/install_directory'
 require 'pathname'

lib/puppet/face/module/list.rb

@@ -1,4 +1,5 @@
 # encoding: UTF-8
+# frozen_string_literal: true
 
 Puppet::Face.define(:module, '1.0.0') do
   action(:list) do

lib/puppet/face/module/uninstall.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 Puppet::Face.define(:module, '1.0.0') do
   action(:uninstall) do
     summary _("Uninstall a puppet module.")

lib/puppet/face/module/upgrade.rb

@@ -1,4 +1,5 @@
 # encoding: UTF-8
+# frozen_string_literal: true
 
 Puppet::Face.define(:module, '1.0.0') do
   action(:upgrade) do