Set Ruby 2.5 as the minimum version

Author: ekohl

.github/workflows/main.yml

@@ -50,6 +50,7 @@ jobs:
     strategy:
       matrix:
         ruby:
+          - '2.5'
           - '2.6'
           - '2.7'
           - '3.0'

vagrant/.rubocop.yml

@@ -6,9 +6,10 @@ AllCops:
   Exclude:
     - !ruby/regexp /katello-installer/
     - !ruby/regexp /.spec/
-  TargetRubyVersion: 2.0
+  TargetRubyVersion: 2.5
+  NewCops: enable
 
-Metrics/LineLength:
+Layout/LineLength:
   Max: 120
 
 Metrics/ClassLength:
@@ -17,14 +18,14 @@ Metrics/ClassLength:
 Metrics/BlockLength:
   Max: 50
 
-Documentation:
+Style/Documentation:
   Enabled: false # don't require documentation
 
-MethodLength:
+Metrics/MethodLength:
   Description: 'Avoid methods longer than 30 lines of code.'
   Max: 30
 
-HashSyntax:
+Style/HashSyntax:
   Enabled: false # don't force 1.9 hash syntax
 
 Layout/EmptyLinesAroundClassBody:

vagrant/Gemfile

@@ -1,21 +1,16 @@
-# rubocop:disable Naming/FileName
 source 'https://rubygems.org'
 
-if RUBY_VERSION < '2.1'
-  gem 'psych', '~> 2.0'
-end
-
 group :test do
   gem 'deep_merge'
   gem 'json'
   gem 'minitest'
   gem 'mocha'
   gem 'rake'
-  if RUBY_VERSION < '3.1'
-    # we need a rubocop that still can manage Ruby 2.0 code
-    # but 0.49.x is not compatible with Ruby 3.1
-    gem 'rubocop', '<0.50'
-  end
-end
 
-# rubocop:enable Naming/FileName
+  # Supports Ruby 2.5
+  # https://docs.rubocop.org/rubocop/compatibility.html#support-matrix
+  gem 'rubocop', '~> 1.28.0'
+  gem 'rubocop-minitest'
+  gem 'rubocop-performance'
+  gem 'rubocop-rake'
+end

vagrant/lib/forklift.rb

@@ -1,8 +1,8 @@
 # frozen_string_literal: true
 
-$LOAD_PATH.unshift File.dirname(__FILE__)
+$LOAD_PATH.unshift __dir__
 
-files = Dir[File.dirname(__FILE__) + '/forklift/**/*.rb']
+files = Dir["#{__dir__}/forklift/**/*.rb"]
 files.uniq.each { |f| require f }
 
 module Forklift

vagrant/lib/forklift/box_distributor.rb

@@ -5,7 +5,7 @@
 module Forklift
   class BoxDistributor
 
-    VAGRANTFILE_API_VERSION = '2'.freeze
+    VAGRANTFILE_API_VERSION = '2'
 
     def initialize(boxes)
       @ansible_groups = {}
@@ -177,12 +177,12 @@ def configure_networks(networks)
     def configure_ansible(machine, ansible, box_name)
       return unless ansible
 
-      if ansible.key?('group') && !ansible['group'].nil?
+      unless ansible['group'].nil?
         @ansible_groups[ansible['group'].to_s] ||= []
         @ansible_groups[ansible['group'].to_s] << box_name
       end
 
-      if ansible.key?('server') && !ansible['server'].nil?
+      unless ansible['server'].nil?
         @ansible_groups["server-#{box_name}"] = ansible['server']
       end
 

vagrant/lib/forklift/box_factory.rb

@@ -31,7 +31,7 @@ def add_boxes!(box_file)
 
     def filter_boxes!
       box_config = Settings.new.settings['boxes']
-      return unless box_config && box_config.key?('exclude')
+      return unless box_config&.key?('exclude')
 
       @boxes.reject! do |name, _box|
         box_config['exclude'].any? { |exclude| name.match(/#{exclude}/) }