Enable Lint/RedundantSafeNavigation rubocop cop

.rubocop.yml

@@ -243,6 +243,9 @@ Lint/RequireParentheses:
 Lint/RedundantStringCoercion:
   Enabled: true
 
+Lint/RedundantSafeNavigation:
+  Enabled: true
+
 Lint/UriEscapeUnescape:
   Enabled: true
 

actionmailbox/lib/action_mailbox/mail_ext/addresses.rb

@@ -32,7 +32,7 @@ def x_forwarded_to_addresses
 
     private
       def address_list(obj)
-        if obj&.respond_to?(:element)
+        if obj.respond_to?(:element)
           # Mail 2.8+
           obj.element
         else

actionpack/lib/action_dispatch/routing/route_set.rb

@@ -860,7 +860,7 @@ def url_for(options, route_name = nil, url_strategy = UNKNOWN, method_name = nil
         params = route_with_params.params
 
         if options.key? :params
-          if options[:params]&.respond_to?(:to_hash)
+          if options[:params].respond_to?(:to_hash)
             params.merge! options[:params]
           else
             params[:params] = options[:params]

activejob/lib/active_job/core.rb

@@ -175,7 +175,7 @@ def set(options = {}) # :nodoc:
     end
 
     def scheduled_at=(value)
-      @_scheduled_at_time = if value&.is_a?(Numeric)
+      @_scheduled_at_time = if value.is_a?(Numeric)
         ActiveJob.deprecator.warn(<<~MSG.squish)
           Assigning a numeric/epoch value to scheduled_at is deprecated. Use a Time object instead.
         MSG

activesupport/lib/active_support/core_ext/time/calculations.rb

@@ -159,7 +159,7 @@ def change(options)
       ::Time.new(new_year, new_month, new_day, new_hour, new_min, new_sec, new_offset)
     elsif utc?
       ::Time.utc(new_year, new_month, new_day, new_hour, new_min, new_sec)
-    elsif zone&.respond_to?(:utc_to_local)
+    elsif zone.respond_to?(:utc_to_local)
       new_time = ::Time.new(new_year, new_month, new_day, new_hour, new_min, new_sec, zone)
 
       # When there are two occurrences of a nominal time due to DST ending,