Upgrade to rubocop 1.48.0 to support ruby 3.2

Remove rubocop-shopify gem

Update .rubocop.yml

Rubocop: auto-correct basic things

Suppress warnings that are time-consuming to fix

Add minor tweaks

Suppress rubocop warnings for redeploy_state and deploy_state

Minor rubocop tweaks

Update file_system_test.rb

Update stack.rb

Update command.rb

Update dev.rake

Author: kartiki975

.rubocop.yml

@@ -13,9 +13,6 @@ AllCops:
     - lib/shipit-engine.rb
     - template.rb
 
-inherit_gem:
-  rubocop-shopify: rubocop.yml
-
 Layout/LineLength:
   Exclude:
     - config/routes.rb
@@ -66,6 +63,41 @@ Style/SymbolArray:
 Naming/InclusiveLanguage:
   Enabled: false
 
+Style/Documentation:
+  Enabled: false
+
+Metrics/ClassLength:
+  Enabled: false
+
+Metrics/BlockLength:
+  Enabled: false
+
+Metrics/ModuleLength:
+    Enabled: false
+
+Metrics/MethodLength:
+  Enabled: false
+
+Naming/MemoizedInstanceVariableName:
+  Enabled: false
+
+Lint/ConstantDefinitionInBlock:
+  Enabled: false
+
+Metrics/AbcSize:
+  Enabled: false
+
+Metrics/CyclomaticComplexity:
+  Enabled: false
+
+Metrics/PerceivedComplexity:
+  Enabled: false
+
+Style/NumericPredicate:
+  Enabled: false
+
+Style/ModuleFunction:
+  Enabled: false
 
 # Crashing cops. We need to update rubocop at some point but better do it in its own PR.
 Layout/BlockAlignment:

CHANGELOG.md

@@ -1,6 +1,7 @@
 # Unreleased
 
 * Pass `Shipit::Stack` to `DeploySpec::FileSystem.new` and make it accessible through an accessor. (#1356)
+* Upgrade Rubocop to 1.48.0
 
 # 0.39.0
 

Gemfile

@@ -13,8 +13,7 @@ end
 group :development, :test do
   gem 'faker'
   gem 'webmock'
-  gem 'rubocop', '1.18.3'
-  gem 'rubocop-shopify', require: false
+  gem 'rubocop', '~> 1.48.0'
 end
 
 group :test do

Gemfile.lock

@@ -109,7 +109,7 @@ GEM
     addressable (2.8.0)
       public_suffix (>= 2.0.2, < 5.0)
     ansi_stream (0.0.6)
-    ast (2.4.2)
+    ast (2.4.3)
     autoprefixer-rails (6.4.1.1)
       execjs
     axiom-types (0.1.1)
@@ -195,6 +195,7 @@ GEM
       rails-dom-testing (>= 1, < 3)
       railties (>= 4.2.0)
       thor (>= 0.14, < 2.0)
+    json (2.10.2)
     jwt (2.7.1)
     lodash-rails (4.17.21)
       railties (>= 3.1)
@@ -252,11 +253,12 @@ GEM
       omniauth (>= 1.9, < 3)
     paquito (0.10.0)
       msgpack (>= 1.5.2)
-    parallel (1.26.3)
-    parser (3.3.4.2)
+    parallel (1.27.0)
+    parser (3.3.8.0)
       ast (~> 2.4.1)
       racc
     pg (1.3.3)
+    prism (1.4.0)
     pry (0.14.1)
       coderay (~> 1.1)
       method_source (~> 1.0)
@@ -318,26 +320,26 @@ GEM
     redis (4.8.1)
     redis-objects (1.7.0)
       redis
-    regexp_parser (2.9.2)
+    regexp_parser (2.10.0)
     reline (0.5.8)
       io-console (~> 0.5)
     responders (3.1.0)
       actionpack (>= 5.2)
       railties (>= 5.2)
     rexml (3.3.9)
-    rubocop (1.18.3)
+    rubocop (1.48.1)
+      json (~> 2.3)
       parallel (~> 1.10)
-      parser (>= 3.0.0.0)
+      parser (>= 3.2.0.0)
       rainbow (>= 2.2.2, < 4.0)
       regexp_parser (>= 1.8, < 3.0)
-      rexml
-      rubocop-ast (>= 1.7.0, < 2.0)
+      rexml (>= 3.2.5, < 4.0)
+      rubocop-ast (>= 1.26.0, < 2.0)
       ruby-progressbar (~> 1.7)
-      unicode-display_width (>= 1.4.0, < 3.0)
-    rubocop-ast (1.32.1)
-      parser (>= 3.3.1.0)
-    rubocop-shopify (2.2.0)
-      rubocop (~> 1.18)
+      unicode-display_width (>= 2.4.0, < 3.0)
+    rubocop-ast (1.44.1)
+      parser (>= 3.3.7.2)
+      prism (~> 1.4)
     ruby-progressbar (1.13.0)
     ruby2_keywords (0.0.5)
     safe_yaml (1.0.5)
@@ -390,7 +392,7 @@ GEM
     timeout (0.4.1)
     tzinfo (2.0.6)
       concurrent-ruby (~> 1.0)
-    unicode-display_width (2.5.0)
+    unicode-display_width (2.6.0)
     useragent (0.16.10)
     validate_url (1.0.15)
       activemodel (>= 3.0.0)
@@ -423,8 +425,7 @@ DEPENDENCIES
   mysql2
   pg
   pry
-  rubocop (= 1.18.3)
-  rubocop-shopify
+  rubocop (~> 1.48.0)
   shipit-engine!
   simplecov
   spy

Rakefile

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 # Add your own tasks in files placed in lib/tasks ending in .rake,
 # for example lib/tasks/capistrano.rake, and they will automatically be available to Rake.
 
@@ -8,7 +9,7 @@ begin
 rescue LoadError
 end
 
-APP_RAKEFILE = File.expand_path('../test/dummy/Rakefile', __FILE__)
+APP_RAKEFILE = File.expand_path('test/dummy/Rakefile', __dir__)
 load('rails/tasks/engine.rake')
 
 Bundler::GemHelper.install_tasks

app/controllers/concerns/shipit/active_model_serializers_patch.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 module Shipit
   module ActiveModelSerializersPatch
     private

app/controllers/concerns/shipit/api/cacheable.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 module Shipit
   module Api
     module Cacheable

app/controllers/concerns/shipit/api/paginable.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 module Shipit
   module Api
     module Paginable
@@ -23,8 +24,8 @@ def render_resources(resource, *)
           resource,
           self,
           order: default_order,
-          max_page_size: max_page_size,
-          default_page_size: default_page_size,
+          max_page_size:,
+          default_page_size:
         )
         headers[LINK] = render_links(paginator.links)
         super(paginator.to_a)

app/controllers/concerns/shipit/api/rendering.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 module Shipit
   module Api
     module Rendering

app/controllers/concerns/shipit/authentication.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 module Shipit
   module Authentication
     extend ActiveSupport::Concern