Layout/EmptyLineAfterMagicComment

This commit enables the Rubocop Layout/EmptyLineAfterMagicComment cop and fixes all offenses.
puppetlabs · Dec 19, 2023 · e7492be · e7492be
1 parent 66d053b
commit e7492be
Show file tree

Hide file tree

Showing 996 changed files with 995 additions and 4 deletions.
diff --git a/.rubocop_todo.yml b/.rubocop_todo.yml
@@ -22,10 +22,6 @@ I18n/GetText/DecorateStringFormattingUsingPercent:
 I18n/RailsI18n/DecorateString:
   Enabled: false
 
-# This cop supports safe auto-correction (--auto-correct).
-Layout/EmptyLineAfterMagicComment:
-  Enabled: false
-
 # This cop supports safe auto-correction (--auto-correct).
 # Configuration parameters: EnforcedStyleAlignWith, Severity.
 # SupportedStylesAlignWith: keyword, variable, start_of_line

diff --git a/lib/hiera/puppet_function.rb b/lib/hiera/puppet_function.rb
@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 require 'hiera_puppet'
 
 # Provides the base class for the puppet functions hiera, hiera_array, hiera_hash, and hiera_include.

diff --git a/lib/hiera/scope.rb b/lib/hiera/scope.rb
@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 require 'forwardable'
 class Hiera
   class Scope

diff --git a/lib/hiera_puppet.rb b/lib/hiera_puppet.rb
@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 Puppet.features.hiera?
 require 'hiera/scope'
 require_relative 'puppet'

diff --git a/lib/puppet.rb b/lib/puppet.rb
@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 require_relative 'puppet/version'
 require_relative 'puppet/concurrent/synchronized'
 

diff --git a/lib/puppet/agent.rb b/lib/puppet/agent.rb
@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 require_relative '../puppet/application'
 require_relative '../puppet/error'
 require_relative '../puppet/util/at_fork'

diff --git a/lib/puppet/agent/disabler.rb b/lib/puppet/agent/disabler.rb
@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 require_relative '../../puppet/util/json_lockfile'
 
 # This module is responsible for encapsulating the logic for

diff --git a/lib/puppet/agent/locker.rb b/lib/puppet/agent/locker.rb
@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 require_relative '../../puppet/util/pidlock'
 require_relative '../../puppet/error'
 

diff --git a/lib/puppet/application.rb b/lib/puppet/application.rb
@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 require 'optparse'
 require_relative '../puppet/util/command_line'
 require_relative '../puppet/util/constant_inflector'

diff --git a/lib/puppet/application/agent.rb b/lib/puppet/application/agent.rb
@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 require_relative '../../puppet/application'
 require_relative '../../puppet/daemon'
 require_relative '../../puppet/util/pidlock'

diff --git a/lib/puppet/application/apply.rb b/lib/puppet/application/apply.rb
@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 require_relative '../../puppet/application'
 require_relative '../../puppet/configurer'
 require_relative '../../puppet/util/profiler/aggregate'

diff --git a/lib/puppet/application/catalog.rb b/lib/puppet/application/catalog.rb
@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 require_relative '../../puppet/application/indirection_base'
 
 class Puppet::Application::Catalog < Puppet::Application::IndirectionBase

diff --git a/lib/puppet/application/config.rb b/lib/puppet/application/config.rb
@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 require_relative '../../puppet/application/face_base'
 
 class Puppet::Application::Config < Puppet::Application::FaceBase

diff --git a/lib/puppet/application/describe.rb b/lib/puppet/application/describe.rb
@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 require_relative '../../puppet/application'
 
 class Formatter

diff --git a/lib/puppet/application/device.rb b/lib/puppet/application/device.rb
@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 require_relative '../../puppet/application'
 require_relative '../../puppet/configurer'
 require_relative '../../puppet/util/network_device'

diff --git a/lib/puppet/application/doc.rb b/lib/puppet/application/doc.rb
@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 require_relative '../../puppet/application'
 
 class Puppet::Application::Doc < Puppet::Application

diff --git a/lib/puppet/application/epp.rb b/lib/puppet/application/epp.rb
@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 require_relative '../../puppet/application/face_base'
 require_relative '../../puppet/face'
 

diff --git a/lib/puppet/application/face_base.rb b/lib/puppet/application/face_base.rb
@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 require_relative '../../puppet/application'
 require_relative '../../puppet/face'
 require 'optparse'

diff --git a/lib/puppet/application/facts.rb b/lib/puppet/application/facts.rb
@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 require_relative '../../puppet/application/indirection_base'
 
 class Puppet::Application::Facts < Puppet::Application::IndirectionBase

diff --git a/lib/puppet/application/filebucket.rb b/lib/puppet/application/filebucket.rb
@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 require_relative '../../puppet/application'
 
 class Puppet::Application::Filebucket < Puppet::Application

diff --git a/lib/puppet/application/generate.rb b/lib/puppet/application/generate.rb
@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 require_relative '../../puppet/application/face_base'
 
 # The Generate application. 

diff --git a/lib/puppet/application/help.rb b/lib/puppet/application/help.rb
@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 require_relative '../../puppet/application/face_base'
 
 class Puppet::Application::Help < Puppet::Application::FaceBase

diff --git a/lib/puppet/application/indirection_base.rb b/lib/puppet/application/indirection_base.rb
@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 require_relative '../../puppet/application/face_base'
 
 class Puppet::Application::IndirectionBase < Puppet::Application::FaceBase

diff --git a/lib/puppet/application/lookup.rb b/lib/puppet/application/lookup.rb
@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 require_relative '../../puppet/application'
 require_relative '../../puppet/pops'
 require_relative '../../puppet/node'

diff --git a/lib/puppet/application/module.rb b/lib/puppet/application/module.rb
@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 require_relative '../../puppet/application/face_base'
 
 class Puppet::Application::Module < Puppet::Application::FaceBase

diff --git a/lib/puppet/application/node.rb b/lib/puppet/application/node.rb
@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 require_relative '../../puppet/application/indirection_base'
 
 class Puppet::Application::Node < Puppet::Application::IndirectionBase

diff --git a/lib/puppet/application/parser.rb b/lib/puppet/application/parser.rb
@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 require_relative '../../puppet/application/face_base'
 require_relative '../../puppet/face'
 

diff --git a/lib/puppet/application/plugin.rb b/lib/puppet/application/plugin.rb
@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 require_relative '../../puppet/application/face_base'
 class Puppet::Application::Plugin < Puppet::Application::FaceBase
   environment_mode :not_required

diff --git a/lib/puppet/application/report.rb b/lib/puppet/application/report.rb
@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 require_relative '../../puppet/application/indirection_base'
 
 class Puppet::Application::Report < Puppet::Application::IndirectionBase

diff --git a/lib/puppet/application/resource.rb b/lib/puppet/application/resource.rb
@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 require_relative '../../puppet/application'
 
 class Puppet::Application::Resource < Puppet::Application

diff --git a/lib/puppet/application/script.rb b/lib/puppet/application/script.rb
@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 require_relative '../../puppet/application'
 require_relative '../../puppet/configurer'
 require_relative '../../puppet/util/profiler/aggregate'

diff --git a/lib/puppet/application/ssl.rb b/lib/puppet/application/ssl.rb
@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 require_relative '../../puppet/application'
 require_relative '../../puppet/ssl/oids'
 

diff --git a/lib/puppet/application_support.rb b/lib/puppet/application_support.rb
@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 require 'yaml'
 
 require_relative '../puppet'

diff --git a/lib/puppet/coercion.rb b/lib/puppet/coercion.rb
@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 # Various methods used to coerce values into a canonical form.
 #
 # @api private

diff --git a/lib/puppet/compilable_resource_type.rb b/lib/puppet/compilable_resource_type.rb
@@ -1,4 +1,5 @@
 # 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

diff --git a/lib/puppet/concurrent.rb b/lib/puppet/concurrent.rb
@@ -1,3 +1,4 @@
 # frozen_string_literal: true
+
 module Puppet::Concurrent
 end
diff --git a/lib/puppet/concurrent/lock.rb b/lib/puppet/concurrent/lock.rb
@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 require_relative '../../puppet/concurrent/synchronized'
 
 module Puppet

diff --git a/lib/puppet/concurrent/synchronized.rb b/lib/puppet/concurrent/synchronized.rb
@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 module Puppet
 module Concurrent
 # Including Puppet::Concurrent::Synchronized into a class when running on JRuby

diff --git a/lib/puppet/concurrent/thread_local_singleton.rb b/lib/puppet/concurrent/thread_local_singleton.rb
@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 module Puppet
   module Concurrent
     module ThreadLocalSingleton

diff --git a/lib/puppet/configurer.rb b/lib/puppet/configurer.rb
@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 # The client for interacting with the puppetmaster config server.
 require 'timeout'
 require_relative '../puppet/util'

diff --git a/lib/puppet/configurer/downloader.rb b/lib/puppet/configurer/downloader.rb
@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 require_relative '../../puppet/configurer'
 require_relative '../../puppet/resource/catalog'
 

diff --git a/lib/puppet/configurer/fact_handler.rb b/lib/puppet/configurer/fact_handler.rb
@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 require_relative '../../puppet/indirector/facts/facter'
 
 require_relative '../../puppet/configurer'

diff --git a/lib/puppet/configurer/plugin_handler.rb b/lib/puppet/configurer/plugin_handler.rb
@@ -1,4 +1,5 @@
 # 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.

diff --git a/lib/puppet/confine.rb b/lib/puppet/confine.rb
@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 # The class that handles testing whether our providers
 # actually work or not.
 require_relative '../puppet/util'

diff --git a/lib/puppet/confine/any.rb b/lib/puppet/confine/any.rb
@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 class Puppet::Confine::Any < Puppet::Confine
   def self.summarize(confines)
     confines.inject(0) { |count, confine| count + confine.summary }

diff --git a/lib/puppet/confine/boolean.rb b/lib/puppet/confine/boolean.rb
@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 require_relative '../../puppet/confine'
 
 # Common module for the Boolean confines. It currently

diff --git a/lib/puppet/confine/exists.rb b/lib/puppet/confine/exists.rb
@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 require_relative '../../puppet/confine'
 
 class Puppet::Confine::Exists < Puppet::Confine

diff --git a/lib/puppet/confine/false.rb b/lib/puppet/confine/false.rb
@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 require_relative '../../puppet/confine/boolean'
 
 class Puppet::Confine::False < Puppet::Confine

diff --git a/lib/puppet/confine/feature.rb b/lib/puppet/confine/feature.rb
@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 require_relative '../../puppet/confine'
 
 class Puppet::Confine::Feature < Puppet::Confine

diff --git a/lib/puppet/confine/true.rb b/lib/puppet/confine/true.rb
@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 require_relative '../../puppet/confine/boolean'
 
 class Puppet::Confine::True < Puppet::Confine

diff --git a/lib/puppet/confine/variable.rb b/lib/puppet/confine/variable.rb
@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 require_relative '../../puppet/confine'
 
 # Require a specific value for a variable, either a Puppet setting

diff --git a/lib/puppet/confine_collection.rb b/lib/puppet/confine_collection.rb
@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 # Manage a collection of confines, returning a boolean or
 # helpful information.
 require_relative '../puppet/confine'

diff --git a/lib/puppet/confiner.rb b/lib/puppet/confiner.rb
@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 require_relative '../puppet/confine_collection'
 
 # The Confiner module contains methods for managing a Provider's confinement (suitability under given

diff --git a/lib/puppet/context.rb b/lib/puppet/context.rb
@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 require_relative '../puppet/thread_local'
 
 # Puppet::Context is a system for tracking services and contextual information

diff --git a/lib/puppet/context/trusted_information.rb b/lib/puppet/context/trusted_information.rb
@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 require_relative '../../puppet/trusted_external'
 
 # @api private

diff --git a/lib/puppet/daemon.rb b/lib/puppet/daemon.rb
@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 require_relative '../puppet/application'
 require_relative '../puppet/scheduler'
 

diff --git a/lib/puppet/data_binding.rb b/lib/puppet/data_binding.rb
@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 require_relative '../puppet/indirector'
 
 # A class for managing data lookups

diff --git a/lib/puppet/datatypes.rb b/lib/puppet/datatypes.rb
@@ -1,4 +1,5 @@
 # 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>'.

diff --git a/lib/puppet/datatypes/error.rb b/lib/puppet/datatypes/error.rb
@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 Puppet::DataTypes.create_type('Error') do
   interface <<-PUPPET
     type_parameters => {

diff --git a/lib/puppet/datatypes/impl/error.rb b/lib/puppet/datatypes/impl/error.rb
@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 class Puppet::DataTypes::Error
   attr_reader :msg, :kind, :issue_code, :details
   alias message msg

diff --git a/lib/puppet/defaults.rb b/lib/puppet/defaults.rb
@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 require_relative '../puppet/util/platform'
 
 module Puppet

diff --git a/lib/puppet/environments.rb b/lib/puppet/environments.rb
@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 require_relative '../puppet/concurrent/synchronized'
 
 # @api private

diff --git a/lib/puppet/error.rb b/lib/puppet/error.rb
@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 module Puppet
   # The base class for all Puppet errors. It can wrap another exception
   class Error < RuntimeError

diff --git a/lib/puppet/etc.rb b/lib/puppet/etc.rb
@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 require_relative '../puppet/util/character_encoding'
 # Wrapper around Ruby Etc module allowing us to manage encoding in a single
 # place.

diff --git a/lib/puppet/external/dot.rb b/lib/puppet/external/dot.rb
@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 # rdot.rb
 #
 #

diff --git a/lib/puppet/face.rb b/lib/puppet/face.rb
@@ -1,4 +1,5 @@
 # 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.