rubocop to latest

Author: onyxraven

.github/workflows/rubocop.yml

@@ -6,22 +6,26 @@ on:
   pull_request:
     branches: [master]
 
+# This allows a subsequently queued workflow run to interrupt previous runs
+concurrency:
+  group: '${{ github.workflow }} @ ${{ github.event.pull_request.head.label || github.head_ref || github.ref }}'
+  cancel-in-progress: true
+
 jobs:
   rubocop:
     name: Rubocop
-    runs-on: ${{ matrix.os }}
+    runs-on: ubuntu-latest
     strategy:
+      fail-fast: false
       matrix:
-        os: [ubuntu-latest]
-        ruby: [
-            2.7
-        ]
+        ruby-version:
+          - "3.3"
 
     steps:
       - uses: actions/checkout@v4
       - uses: ruby/setup-ruby@v1
         with:
-          ruby-version: ${{ matrix.ruby }}
+          ruby-version: ${{ matrix.ruby-version }}
           bundler-cache: true
       - name: Ruby linter
         run: bundle exec rubocop

.github/workflows/ruby.yml

@@ -35,13 +35,13 @@ jobs:
           - "3.2"
           - "3.3"
         resque-version:
-          # - "master"
-          - "~> 2.6"
+          - "master"
+          - "~> 2.4"
           - "~> 1.27"
         rufus-scheduler:
-          # - "~> 3.6.0"
-          # - "~> 3.7.0"
-          # - "~> 3.8.0"
+          - "~> 3.6.0"
+          - "~> 3.7.0"
+          - "~> 3.8.0"
           - "~> 3.9"
         redis-version:
           - "~> 3.3"

.rubocop.yml

@@ -1,18 +1,22 @@
-inherit_from: .rubocop_todo.yml
-
 AllCops:
+  SuggestExtensions: false
+  TargetRubyVersion: 3.0
+  NewCops: enable
   Include:
   - Gemfile
   - '**/Rakefile'
   - resque-scheduler.gemspec
   - bin/resque-scheduler
 
-Documentation:
+Gemspec/DevelopmentDependencies:
+  Enabled: false
+Metrics/ClassLength:
+  Max: 110
+Metrics/PerceivedComplexity:
+  Enabled: false
+Naming/HeredocDelimiterNaming:
   Enabled: false
-
 Style/DoubleNegation:
   Enabled: false
-Metrics/PerceivedComplexity:
+Style/FrozenStringLiteralComment:
   Enabled: false
-Metrics/ClassLength:
-  Max: 110

.rubocop_todo.yml

@@ -1,27 +1,27 @@
 # vim:fileencoding=utf-8
-lib = File.expand_path('../lib', __FILE__)
+lib = File.expand_path('lib', __dir__)
 $LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
 require 'resque/scheduler/version'
 
 Gem::Specification.new do |spec|
   spec.name = 'resque-scheduler'
   spec.version = Resque::Scheduler::VERSION
-  spec.authors = <<-EOF.split(/\n/).map(&:strip)
+  spec.authors = <<-EOF.split("\n").map(&:strip)
     Ben VandenBos
     Simon Eskildsen
     Ryan Biesemeyer
     Dan Buch
     Michael Bianco
     Patrick Tulskie
   EOF
-  spec.email = %w(
+  spec.email = %w[
     bvandenbos@gmail.com
     sirup@sirupsen.com
     ryan@yaauie.com
     dan@meatballhat.com
     mike@mikebian.co
     patricktulskie@gmail.com
-  )
+  ]
   spec.summary = 'Light weight job scheduling on top of Resque'
   spec.description = <<-DESCRIPTION
     Light weight job scheduling on top of Resque.
@@ -32,7 +32,7 @@ Gem::Specification.new do |spec|
   spec.license = 'MIT'
   spec.metadata['rubygems_mfa_required'] = 'true'
 
-  spec.required_ruby_version = '>= 2.3.0'
+  spec.required_ruby_version = '>= 3.0'
 
   spec.files = `git ls-files -z`.split("\0").reject do |f|
     f.match(%r{^(test|spec|features|examples|bin|tasks)/}) ||
@@ -41,28 +41,28 @@ Gem::Specification.new do |spec|
   end
   spec.bindir = 'exe'
   spec.executables = spec.files.grep(%r{^exe/}) { |f| File.basename(f) }
-  spec.require_paths = %w(lib)
+  spec.require_paths = %w[lib]
 
   spec.add_development_dependency 'bundler'
   spec.add_development_dependency 'json'
   spec.add_development_dependency 'minitest'
   spec.add_development_dependency 'mocha'
+  spec.add_development_dependency 'ostruct'
   spec.add_development_dependency 'pry'
+  spec.add_development_dependency 'racc'
   spec.add_development_dependency 'rack-test'
   spec.add_development_dependency 'rake'
   spec.add_development_dependency 'simplecov'
   spec.add_development_dependency 'test-unit'
-  spec.add_development_dependency 'yard'
   spec.add_development_dependency 'timecop'
+  spec.add_development_dependency 'yard'
 
-  # We pin rubocop because new cops have a tendency to result in false-y
-  # positives for new contributors, which is not a nice experience.
-  spec.add_development_dependency 'rubocop', '~> 0.40.0'
+  spec.add_development_dependency 'rubocop', '~> 1.4'
 
-  spec.add_runtime_dependency 'mono_logger', '~> 1.0'
-  spec.add_runtime_dependency 'redis', '>= 3.3'
-  spec.add_runtime_dependency 'resque', '>= 1.27'
+  spec.add_dependency 'mono_logger', '~> 1.0'
+  spec.add_dependency 'redis', '>= 3.3'
+  spec.add_dependency 'resque', '>= 1.27'
   # rufus-scheduler v3.7 causes a failure in test/multi_process_test.rb
   # rufus-scheduler v3.3 is missing a to_local method which fails tests
-  spec.add_runtime_dependency 'rufus-scheduler', '~> 3.2', '!= 3.3'
+  spec.add_dependency 'rufus-scheduler', '~> 3.2', '!= 3.3'
 end