[ci] Fix EmptyLinesAroundExceptionHandlingKeywords

We fix the Layou/EmptyLinesAroundExceptionHandlingKeywords cop complains.
openSUSE · Dec 12, 2017 · 24134a6 · 24134a6
1 parent f0a335a
commit 24134a6
Show file tree

Hide file tree

Showing 6 changed files with 0 additions and 18 deletions.
diff --git a/.rubocop_todo.yml b/.rubocop_todo.yml
@@ -13,16 +13,6 @@
 Layout/DotPosition:
   Enabled: false
 
-# Offense count: 8
-# Cop supports --auto-correct.
-Layout/EmptyLinesAroundExceptionHandlingKeywords:
-  Exclude:
-    - 'src/api/app/controllers/webui/package_controller.rb'
-    - 'src/api/app/controllers/webui/project_controller.rb'
-    - 'src/api/app/controllers/webui/request_controller.rb'
-    - 'src/api/app/controllers/webui/subscriptions_controller.rb'
-    - 'src/api/app/controllers/webui/users/subscriptions_controller.rb'
-
 # Offense count: 13
 # Cop supports --auto-correct.
 # Configuration parameters: AllowForAlignment, ForceEqualSignAlignment.

diff --git a/src/api/app/controllers/webui/package_controller.rb b/src/api/app/controllers/webui/package_controller.rb
@@ -565,7 +565,6 @@ def branch
       flash[:notice] = 'Successfully branched package'
       redirect_to(package_show_path(project: created_project_name, package: created_package_name))
     end
-
   rescue BranchPackage::DoubleBranchPackageError => exception
     flash[:notice] = 'You have already branched this package'
     redirect_to(package_show_path(project: exception.project, package: exception.package))
@@ -840,10 +839,8 @@ def update_build_log
         @log_chunk = chunk
         @offset += ActiveXML.backend.last_body_length
       end
-
     rescue Timeout::Error, IOError
       @log_chunk = ''
-
     rescue ActiveXML::Transport::Error => e
       if e.summary =~ %r{Logfile is not that big}
         @log_chunk = ''
@@ -1001,7 +998,6 @@ def save_meta
       else
         errors << "Package doesn't exists in that project."
       end
-
     rescue Suse::ValidationError => e
       errors << e.message
     end

diff --git a/src/api/app/controllers/webui/project_controller.rb b/src/api/app/controllers/webui/project_controller.rb
@@ -565,7 +565,6 @@ def save_meta
           @project.store if errors.empty?
         end
       end
-
     rescue Suse::ValidationError => exception
       errors << exception.message
     end

diff --git a/src/api/app/controllers/webui/request_controller.rb b/src/api/app/controllers/webui/request_controller.rb
@@ -204,7 +204,6 @@ def delete_request
 
       request_link = ActionController::Base.helpers.link_to("repository delete request #{req.number}", request_show_path(req.number))
       flash[:success] = "Created #{request_link}"
-
     rescue APIException => e
       flash[:error] = e.message
       if params[:package]

diff --git a/src/api/app/controllers/webui/subscriptions_controller.rb b/src/api/app/controllers/webui/subscriptions_controller.rb
@@ -8,7 +8,6 @@ def index
   def update
     subscriptions_form.update!(params[:subscriptions])
     flash[:notice] = 'Notifications settings updated'
-
   rescue ActiveRecord::RecordInvalid
     flash[:error] = 'Notifications settings could not be updated due to an error'
   ensure

diff --git a/src/api/app/controllers/webui/users/subscriptions_controller.rb b/src/api/app/controllers/webui/users/subscriptions_controller.rb
@@ -16,7 +16,6 @@ def update
 
     subscriptions_form.update!(params[:subscriptions])
     flash[:notice] = 'Notifications settings updated'
-
   rescue ActiveRecord::RecordInvalid
     flash[:error] = 'Notifications settings could not be updated due to an error'
   ensure