[ci][frontend] Enable three more rubocops

* Layout/SpaceAroundBlockParameters * Layout/SpaceAroundKeyword * Layout/SpaceBeforeComma Updated via rubocop's --auto-correct
openSUSE · Oct 12, 2017 · d3c072e · d3c072e
1 parent 28826fa
commit d3c072e
Show file tree

Hide file tree

Showing 4 changed files with 4 additions and 25 deletions.
diff --git a/.rubocop_todo.yml b/.rubocop_todo.yml
@@ -248,34 +248,13 @@ Layout/MultilineMethodCallIndentation:
 Layout/MultilineOperationIndentation:
   Enabled: false
 
-# Offense count: 6
-# Cop supports --auto-correct.
-# Configuration parameters: EnforcedStyleInsidePipes, SupportedStylesInsidePipes.
-# SupportedStylesInsidePipes: space, no_space
-Layout/SpaceAroundBlockParameters:
-  Exclude:
-    - 'src/api/app/views/result/packageresult.xml.builder'
-    - 'src/api/app/views/result/projectresult.xml.builder'
-
-# Offense count: 1
-# Cop supports --auto-correct.
-Layout/SpaceAroundKeyword:
-  Exclude:
-    - 'src/api/app/views/person/userinfo.xml.builder'
-
 # Offense count: 68
 # Cop supports --auto-correct.
 # Configuration parameters: EnforcedStyle, SupportedStyles.
 # SupportedStyles: space, no_space
 Layout/SpaceBeforeBlockBraces:
   Enabled: false
 
-# Offense count: 1
-# Cop supports --auto-correct.
-Layout/SpaceBeforeComma:
-  Exclude:
-    - 'src/api/app/views/tag/_tagged_objects_with_tags.xml.builder'
-
 # Offense count: 4
 # Cop supports --auto-correct.
 # Configuration parameters: EnforcedStyle, SupportedStyles.

diff --git a/src/api/app/views/person/userinfo.xml.builder b/src/api/app/views/person/userinfo.xml.builder
@@ -3,7 +3,7 @@ xml.person do
   xml.email @render_user.email
   xml.realname @render_user.realname
 
-  if( @render_user.watched_projects.count > 0 )
+  if @render_user.watched_projects.count > 0
     xml.watchlist do
       @render_user.watched_projects.each do |wp|
         xml.project :name => wp.name

diff --git a/src/api/app/views/result/packageresult.xml.builder b/src/api/app/views/result/packageresult.xml.builder
@@ -5,7 +5,7 @@ xml.packageresult( "project" => @project, "repository" => @repository,
     xml.packagecount( @succeeded, "state" => "succeeded" )
     xml.packagecount( @failed, "state" => "failed" )
   end
-  @arch_status.each do | a, s |
+  @arch_status.each do |a, s|
     xml.archresult( "arch" => a ) do
       xml.status( "code" => s["code"] ) do
         if ( s["summary"] )

diff --git a/src/api/app/views/result/projectresult.xml.builder b/src/api/app/views/result/projectresult.xml.builder
@@ -6,9 +6,9 @@ xml.projectresult( "project" => @project ) do
     xml.packagecount( @building, "state" => "building" )
     xml.packagecount( @delayed, "state" => "delayed" )
   end
-  @repository_status.each do | r, arch_status |
+  @repository_status.each do |r, arch_status|
     xml.repositoryresult( "name" => r ) do
-      arch_status.each do | a, s | 
+      arch_status.each do |a, s|
         xml.archresult( "arch" => a ) do
           xml.status( "code" => s )
         end