Suppress rubocop warnings for redeploy_state and deploy_state

Author: kartiki975

app/models/shipit/deploy.rb

@@ -94,7 +94,7 @@ def build_rollback(user = nil, env: nil, force: false)
         parent_id: id,
         since_commit: stack.last_deployed_commit,
         until_commit:,
-        env: env&.to_h || {},
+        env: env.to_h,
         allow_concurrency: force,
         ignored_safeties: force,
         max_retries: stack.retries_on_rollback

app/models/shipit/stack.rb

@@ -132,7 +132,7 @@ def undeployed_commits?
 
     def trigger_task(definition_id, user, env: nil, force: false)
       definition = find_task_definition(definition_id)
-      env = env&.to_h || {}
+      env = env.to_h
 
       definition.variables_with_defaults.each do |variable|
         env[variable.name] ||= variable.default
@@ -158,7 +158,7 @@ def build_deploy(until_commit, user, env: nil, force: false, allow_concurrency:
         user_id: user.id,
         until_commit:,
         since_commit:,
-        env: filter_deploy_envs(env&.to_h || {}),
+        env: filter_deploy_envs(env.to_h),
         allow_concurrency:,
         ignored_safeties: force || !until_commit.deployable?,
         max_retries: retries_on_deploy

app/models/shipit/task.rb

@@ -51,7 +51,7 @@ def dump(hash)
         end
 
         def load(hash)
-          hash&.to_h || {} # cast back to a real hash
+          hash.to_h # cast back to a real hash
         end
 
         def new

app/models/shipit/undeployed_commit.rb

@@ -1,5 +1,10 @@
 # frozen_string_literal: true
 
+# rubocop:disable Lint/MissingCopEnableDirective, Style/OptionalBooleanParameter
+# Disabling for now because we need to support the `bypass_safeties` parameter
+# in the `deploy_state` and `redeploy_state` methods. We can revisit this later.
+
+require 'delegate'
 module Shipit
   class UndeployedCommit < DelegateClass(Commit)
     attr_reader :index