Enable Style/IdenticalConditionalBranches Rubocop cop

.rubocop.yml

@@ -284,7 +284,7 @@ Style/IfWithSemicolon:
 # Checks that conditional statements do not have an identical line at the
 # end of each branch, which can validly be moved out of the conditional.
 Style/IdenticalConditionalBranches:
-  Enabled: false
+  Enabled: true
 
 # Checks the indentation of the first line of the right-hand-side of a
 # multi-line assignment.

app/controllers/projects_controller.rb

@@ -53,11 +53,11 @@ class ProjectsController < Projects::ApplicationController
             notice: "Project '#{@project.name}' was successfully updated."
           )
         end
-        format.js
       else
         format.html { render 'edit' }
-        format.js
       end
+
+      format.js
     end
   end
 

app/models/project_services/irker_service.rb

@@ -114,13 +114,13 @@ class IrkerService < Service
     if uri.is_a?(URI) && uri.scheme[/^ircs?\z/] && !uri.path.nil?
       # Do not authorize irc://domain.com/
       if uri.fragment.nil? && uri.path.length > 1
-        uri.to_s
       else
         # Authorize irc://domain.com/smthg#chan
         # The irker daemon will deal with it by concatenating smthg and
         # chan, thus sending messages on #smthgchan
-        uri.to_s
       end
+
+      uri.to_s
     end
   end
 end

app/services/merge_requests/refresh_service.rb

@@ -61,19 +61,14 @@ module MergeRequests
       merge_requests.each do |merge_request|
         if merge_request.source_branch == @branch_name || force_push?
           merge_request.reload_diff
-          merge_request.mark_as_unchecked
         else
           mr_commit_ids = merge_request.commits.map(&:id)
           push_commit_ids = @commits.map(&:id)
           matches = mr_commit_ids & push_commit_ids
-
+          merge_request.reload_diff if matches.any?
-          if matches.any?
-            merge_request.reload_diff
-            merge_request.mark_as_unchecked
-          else
-            merge_request.mark_as_unchecked
-          end
         end
+
+        merge_request.mark_as_unchecked
       end
     end
 

lib/rouge/formatters/html_gitlab.rb

@@ -143,18 +143,14 @@ module Rouge
               '</span>'
             end
           end
-          lines.join("\n")
+        elsif @linenos == 'inline'
-        else
+          lines = lines.each_with_index.map do |line, index|
-          if @linenos == 'inline'
+            number = index + @linenostart
-            lines = lines.each_with_index.map do |line, index|
+            "<span class=\"linenos\">#{number}</span>#{line}"
-              number = index + @linenostart
-              "<span class=\"linenos\">#{number}</span>#{line}"
-            end
-            lines.join("\n")
-          else
-            lines.join("\n")
           end
         end
+
+        lines.join("\n")
       end
 
       def span(tok, val)