Merge branch 'ee-56392-enable-the-layout-extraspacing-cop' into 'master'

[EE] Enable the Layout/ExtraSpacing cop

Closes gitlab-ce#56392

See merge request gitlab-org/gitlab-ee!9290

.rubocop_todo.yml

@@ -15,12 +15,6 @@ Capybara/CurrentPathExpectation:
 Layout/EmptyLinesAroundArguments:
   Enabled: false
 
-# Offense count: 253
-# Cop supports --auto-correct.
-# Configuration parameters: AllowForAlignment, ForceEqualSignAlignment.
-Layout/ExtraSpacing:
-  Enabled: false
-
 # Offense count: 83
 # Cop supports --auto-correct.
 # Configuration parameters: EnforcedStyle, IndentationWidth.

app/controllers/passwords_controller.rb

@@ -5,7 +5,7 @@ class PasswordsController < Devise::PasswordsController
 
   before_action :resource_from_email, only: [:create]
   before_action :check_password_authentication_available, only: [:create]
-  before_action :throttle_reset,      only: [:create]
+  before_action :throttle_reset, only: [:create]
 
   # rubocop: disable CodeReuse/ActiveRecord
   def edit

app/controllers/projects/commit_controller.rb

@@ -24,10 +24,10 @@ class Projects::CommitController < Projects::ApplicationController
     apply_diff_view_cookie!
 
     respond_to do |format|
-      format.html  do
+      format.html do
         render
       end
-      format.diff  do
+      format.diff do
         send_git_diff(@project.repository, @commit.diff_refs)
       end
       format.patch do

app/controllers/projects/discussions_controller.rb

@@ -40,7 +40,7 @@ class Projects::DiscussionsController < Projects::ApplicationController
 
   def render_json_with_discussions_serializer
     render json:
-      DiscussionSerializer.new(project: project, noteable: discussion.noteable, current_user: current_user, note_entity:  ProjectNoteEntity)
+      DiscussionSerializer.new(project: project, noteable: discussion.noteable, current_user: current_user, note_entity: ProjectNoteEntity)
       .represent(discussion, context: self, render_truncated_diff_lines: true)
   end
 

app/controllers/projects/merge_requests_controller.rb

@@ -55,7 +55,7 @@ class Projects::MergeRequestsController < Projects::MergeRequests::ApplicationCo
         render json: serializer.represent(@merge_request, serializer: params[:serializer])
       end
 
-      format.patch  do
+      format.patch do
         break render_404 unless @merge_request.diff_refs
 
         send_git_patch @project.repository, @merge_request.diff_refs

app/controllers/projects/pages_controller.rb

@@ -18,7 +18,7 @@ class Projects::PagesController < Projects::ApplicationController
     project.pages_domains.destroy_all # rubocop: disable DestroyAll
 
     respond_to do |format|
-      format.html  do
+      format.html do
         redirect_to project_pages_path(@project),
                     status: 302,
                     notice: 'Pages were removed'

app/graphql/types/permission_types/project.rb

@@ -8,7 +8,7 @@ module Types
       abilities :change_namespace, :change_visibility_level, :rename_project,
                 :remove_project, :archive_project, :remove_fork_project,
                 :remove_pages, :read_project, :create_merge_request_in,
-                :read_wiki, :read_project_member, :create_issue,  :upload_file,
+                :read_wiki, :read_project_member, :create_issue, :upload_file,
                 :read_cycle_analytics, :download_code, :download_wiki_code,
                 :fork_project, :create_project_snippet, :read_commit_status,
                 :request_access, :create_pipeline, :create_pipeline_schedule,

app/helpers/auth_helper.rb

@@ -31,7 +31,7 @@ module AuthHelper
   def form_based_provider_with_highest_priority
     @form_based_provider_with_highest_priority ||= begin
       form_based_provider_priority.each do |provider_regexp|
-        highest_priority = form_based_providers.find {  |provider| provider.match?(provider_regexp) }
+        highest_priority = form_based_providers.find { |provider| provider.match?(provider_regexp) }
         break highest_priority unless highest_priority.nil?
       end
     end

app/helpers/issuables_helper.rb

@@ -209,7 +209,7 @@ module IssuablesHelper
   end
 
   def issuable_labels_tooltip(labels, limit: 5)
-    first, last = labels.partition.with_index { |_, i| i < limit  }
+    first, last = labels.partition.with_index { |_, i| i < limit }
 
     if labels && labels.any?
       label_names = first.collect { |label| label.fetch(:title) }

app/models/clusters/platforms/kubernetes.rb

@@ -154,7 +154,7 @@ module Clusters
 
       def build_kube_client!
         raise "Incomplete settings" unless api_url
-        raise "No namespace" if cluster.project_type? && actual_namespace.empty?  # can probably remove this line once we remove #actual_namespace
+        raise "No namespace" if cluster.project_type? && actual_namespace.empty? # can probably remove this line once we remove #actual_namespace
 
         unless (username && password) || token
           raise "Either username/password or token is required to access API"

app/models/concerns/has_status.rb

@@ -85,11 +85,11 @@ module HasStatus
     scope :running, -> { where(status: 'running') }
     scope :pending, -> { where(status: 'pending') }
     scope :success, -> { where(status: 'success') }
-    scope :failed, -> { where(status: 'failed')  }
-    scope :canceled, -> { where(status: 'canceled')  }
-    scope :skipped, -> { where(status: 'skipped')  }
-    scope :manual, -> { where(status: 'manual')  }
-    scope :scheduled, -> { where(status: 'scheduled')  }
+    scope :failed, -> { where(status: 'failed') }
+    scope :canceled, -> { where(status: 'canceled') }
+    scope :skipped, -> { where(status: 'skipped') }
+    scope :manual, -> { where(status: 'manual') }
+    scope :scheduled, -> { where(status: 'scheduled') }
     scope :alive, -> { where(status: [:created, :pending, :running]) }
     scope :created_or_pending, -> { where(status: [:created, :pending]) }
     scope :running_or_pending, -> { where(status: [:running, :pending]) }

app/models/email.rb

@@ -15,7 +15,7 @@ class Email < ActiveRecord::Base
   after_commit :update_invalid_gpg_signatures, if: -> { previous_changes.key?('confirmed_at') }
 
   devise :confirmable
-  self.reconfirmable = false  # currently email can't be changed, no need to reconfirm
+  self.reconfirmable = false # currently email can't be changed, no need to reconfirm
 
   delegate :username, to: :user
 

app/models/group.rb

@@ -98,7 +98,7 @@ class Group < Namespace
     def select_for_project_authorization
       if current_scope.joins_values.include?(:shared_projects)
         joins('INNER JOIN namespaces project_namespace ON project_namespace.id = projects.namespace_id')
-          .where('project_namespace.share_with_group_lock = ?',  false)
+          .where('project_namespace.share_with_group_lock = ?', false)
           .select("projects.id AS project_id, LEAST(project_group_links.group_access, members.access_level) AS access_level")
       else
         super

app/models/member.rb

@@ -76,7 +76,7 @@ class Member < ActiveRecord::Base
   scope :maintainers, -> { active.where(access_level: MAINTAINER) }
   scope :masters, -> { maintainers } # @deprecated
   scope :owners,  -> { active.where(access_level: OWNER) }
-  scope :owners_and_maintainers,  -> { active.where(access_level: [OWNER, MAINTAINER]) }
+  scope :owners_and_maintainers, -> { active.where(access_level: [OWNER, MAINTAINER]) }
   scope :owners_and_masters,  -> { owners_and_maintainers } # @deprecated
 
   scope :order_name_asc, -> { left_join_users.reorder(Gitlab::Database.nulls_last_order('users.name', 'ASC')) }

app/models/milestone.rb

@@ -30,7 +30,7 @@ class Milestone < ActiveRecord::Base
   has_internal_id :iid, scope: :group, init: ->(s) { s&.group&.milestones&.maximum(:iid) }
 
   has_many :issues
-  has_many :labels, -> { distinct.reorder('labels.title') },  through: :issues
+  has_many :labels, -> { distinct.reorder('labels.title') }, through: :issues
   has_many :merge_requests
   has_many :events, as: :target, dependent: :destroy # rubocop:disable Cop/ActiveRecordDependent
 

app/models/project_services/irker_service.rb

@@ -59,7 +59,7 @@ class IrkerService < Service
         ' append "?key=secretpassword" to the URI (Note that due to a bug, if you ' \
         ' want to use a password, you have to omit the "#" on the channel). If you ' \
         ' specify a default IRC URI to prepend before each recipient, you can just ' \
-        ' give a channel name.'  },
+        ' give a channel name.' },
       { type: 'checkbox', name: 'colorize_messages' }
     ]
   end

app/models/snippet.rb

@@ -50,11 +50,11 @@ class Snippet < ActiveRecord::Base
   validates :visibility_level, inclusion: { in: Gitlab::VisibilityLevel.values }
 
   # Scopes
-  scope :are_internal,  -> { where(visibility_level: Snippet::INTERNAL) }
+  scope :are_internal, -> { where(visibility_level: Snippet::INTERNAL) }
   scope :are_private, -> { where(visibility_level: Snippet::PRIVATE) }
   scope :are_public, -> { where(visibility_level: Snippet::PUBLIC) }
   scope :public_and_internal, -> { where(visibility_level: [Snippet::PUBLIC, Snippet::INTERNAL]) }
-  scope :fresh,   -> { order("created_at DESC") }
+  scope :fresh, -> { order("created_at DESC") }
   scope :inc_relations_for_view, -> { includes(author: :status) }
 
   participant :author

app/models/user.rb

@@ -145,7 +145,7 @@ class User < ActiveRecord::Base
 
   has_many :issue_assignees
   has_many :assigned_issues, class_name: "Issue", through: :issue_assignees, source: :issue
-  has_many :assigned_merge_requests,  dependent: :nullify, foreign_key: :assignee_id, class_name: "MergeRequest" # rubocop:disable Cop/ActiveRecordDependent
+  has_many :assigned_merge_requests, dependent: :nullify, foreign_key: :assignee_id, class_name: "MergeRequest" # rubocop:disable Cop/ActiveRecordDependent
 
   has_many :custom_attributes, class_name: 'UserCustomAttribute'
   has_many :callouts, class_name: 'UserCallout'

app/serializers/pipeline_entity.rb

@@ -59,7 +59,7 @@ class PipelineEntity < Grape::Entity
     pipeline.present.failure_reason
   end
 
-  expose :retry_path, if: -> (*) { can_retry? }  do |pipeline|
+  expose :retry_path, if: -> (*) { can_retry? } do |pipeline|
     retry_project_pipeline_path(pipeline.project, pipeline)
   end
 

app/services/boards/issues/move_service.rb

@@ -50,7 +50,7 @@ module Boards
 
         if move_between_ids
           attrs[:move_between_ids] = move_between_ids
-          attrs[:board_group_id] =  board.group&.id
+          attrs[:board_group_id] = board.group&.id
         end
 
         attrs

app/services/boards/lists/destroy_service.rb

@@ -20,7 +20,7 @@ module Boards
 
       # rubocop: disable CodeReuse/ActiveRecord
       def decrement_higher_lists(list)
-        board.lists.movable.where('position > ?',  list.position)
+        board.lists.movable.where('position > ?', list.position)
                    .update_all('position = position - 1')
       end
       # rubocop: enable CodeReuse/ActiveRecord

app/services/labels/promote_service.rb

@@ -49,7 +49,7 @@ module Labels
         .new(current_user, title: new_label.title, group_id: project.group.id)
         .execute(skip_authorization: true)
         .where.not(id: new_label)
-        .select(:id)  # Can't use pluck() to avoid object-creation because of the batching
+        .select(:id) # Can't use pluck() to avoid object-creation because of the batching
     end
     # rubocop: enable CodeReuse/ActiveRecord
 

app/views/admin/groups/show.html.haml

@@ -104,7 +104,7 @@
             - link_to_help = link_to(_("here"), help_page_path("user/permissions"))
             = _('Read more about project permissions <strong>%{link_to_help}</strong>').html_safe % { link_to_help: link_to_help }
 
-          = form_tag admin_group_members_update_path(@group), id: "new_project_member", class: "bulk_import", method: :put  do
+          = form_tag admin_group_members_update_path(@group), id: "new_project_member", class: "bulk_import", method: :put do
             %div
               = users_select_tag(:user_ids, multiple: true, email_user: true, skip_ldap: @group.ldap_synced?, scope: :all)
             .prepend-top-10

app/views/ci/status/_dropdown_graph_badge.html.haml

@@ -6,7 +6,7 @@
 - tooltip = "#{subject.name} - #{status.status_tooltip}"
 
 - if status.has_details?
-  = link_to status.details_path, class: 'mini-pipeline-graph-dropdown-item', data: { toggle: 'tooltip', title: tooltip, container: 'body' }  do
+  = link_to status.details_path, class: 'mini-pipeline-graph-dropdown-item', data: { toggle: 'tooltip', title: tooltip, container: 'body' } do
     %span{ class: klass }= sprite_icon(status.icon)
     %span.ci-build-text= subject.name
 
@@ -16,5 +16,5 @@
     %span.ci-build-text= subject.name
 
 - if status.has_action?
-  = link_to status.action_path, class: "ci-action-icon-container ci-action-icon-wrapper js-ci-action-icon", method: status.action_method, data: { toggle: 'tooltip', title: status.action_title, container: 'body' }  do
+  = link_to status.action_path, class: "ci-action-icon-container ci-action-icon-wrapper js-ci-action-icon", method: status.action_method, data: { toggle: 'tooltip', title: status.action_title, container: 'body' } do
     = sprite_icon(status.action_icon, css_class: "icon-action-#{status.action_icon}")

app/views/dashboard/groups/index.html.haml

 - @hide_top_links = true
 - page_title "Groups"
-- header_title  "Groups", dashboard_groups_path
+- header_title "Groups", dashboard_groups_path
 
 = render_if_exists "shared/gold_trial_callout"
 = render 'dashboard/groups_head'

app/views/devise/passwords/edit.html.haml

@@ -7,7 +7,7 @@
       = f.hidden_field :reset_password_token
       .form-group
         = f.label 'New password', for: "user_password"
-        = f.password_field :password, class: "form-control top qa-password-field",  required: true, title: 'This field is required'
+        = f.password_field :password, class: "form-control top qa-password-field", required: true, title: 'This field is required'
       .form-group
         = f.label 'Confirm new password', for: "user_password_confirmation"
         = f.password_field :password_confirmation, class: "form-control bottom qa-password-confirmation", title: 'This field is required', required: true

app/views/devise/sessions/two_factor.html.haml

@@ -7,7 +7,7 @@
           - resource_params = params[resource_name].presence || params
           = f.hidden_field :remember_me, value: resource_params.fetch(:remember_me, 0)
           %div
-            = f.label 'Two-Factor Authentication code', name:  :otp_attempt
+            = f.label 'Two-Factor Authentication code', name: :otp_attempt
             = f.text_field :otp_attempt, class: 'form-control', required: true, autofocus: true, autocomplete: 'off', title: 'This field is required.'
             %p.form-text.text-muted.hint Enter the code from the two-factor app on your mobile device. If you've lost your device, you may enter one of your recovery codes.
             .prepend-top-20

app/views/doorkeeper/authorizations/new.html.haml

@@ -31,7 +31,7 @@
                   %strong= t scope, scope: [:doorkeeper, :scopes]
                   .text-secondary= t scope, scope: [:doorkeeper, :scope_desc]
         .form-actions.text-right
-          = form_tag oauth_authorization_path, method: :delete, class: 'inline'  do
+          = form_tag oauth_authorization_path, method: :delete, class: 'inline' do
             = hidden_field_tag :client_id, @pre_auth.client.uid
             = hidden_field_tag :redirect_uri, @pre_auth.redirect_uri
             = hidden_field_tag :state, @pre_auth.state

app/views/events/event/_note.html.haml

@@ -21,6 +21,6 @@
         = link_to note.attachment.url, target: '_blank' do
           = image_tag note.attachment.url, class: 'note-image-attach'
       - else
-        = link_to note.attachment.url, target: '_blank',  class: 'note-file-attach' do
+        = link_to note.attachment.url, target: '_blank', class: 'note-file-attach' do
           %i.fa.fa-paperclip
           = note.attachment_identifier

app/views/groups/group_members/index.html.haml

@@ -25,7 +25,7 @@
         Members with access to
         %strong= @group.name
       %span.badge= @members.total_count
-      = form_tag group_group_members_path(@group), method: :get, class: 'form-inline member-search-form flex-project-members-form'  do
+      = form_tag group_group_members_path(@group), method: :get, class: 'form-inline member-search-form flex-project-members-form' do
         .form-group
           .position-relative.append-right-8
             = search_field_tag :search, params[:search], { placeholder: 'Find existing members by name', class: 'form-control', spellcheck: false }

app/views/groups/merge_requests.html.haml

@@ -7,7 +7,7 @@
     = render 'shared/issuable/nav', type: :merge_requests
     - if current_user
       .nav-controls
-        = render 'shared/new_project_item_select', path: 'merge_requests/new', label: "New merge request",  type: :merge_requests, with_feature_enabled: 'merge_requests', with_shared: false, include_projects_in_subgroups: true
+        = render 'shared/new_project_item_select', path: 'merge_requests/new', label: "New merge request", type: :merge_requests, with_feature_enabled: 'merge_requests', with_shared: false, include_projects_in_subgroups: true
 
   = render 'shared/issuable/search_bar', type: :merge_requests
 

app/views/groups/milestones/_form.html.haml

-= form_for [@group, @milestone], html: { class: 'milestone-form common-note-form js-quick-submit js-requires-input' }  do |f|
+= form_for [@group, @milestone], html: { class: 'milestone-form common-note-form js-quick-submit js-requires-input' } do |f|
   .row
     = form_errors(@milestone)
 
@@ -24,4 +24,3 @@
     - else
       = f.submit 'Update milestone', class: "btn-success btn"
       = link_to "Cancel", group_milestone_path(@group, @milestone), class: "btn btn-cancel"
-

app/views/layouts/_head.html.haml

@@ -11,20 +11,20 @@
   %meta{ 'http-equiv' => 'X-UA-Compatible', content: 'IE=edge' }
 
   -# Open Graph - http://ogp.me/
-  %meta{ property: 'og:type',        content: "object" }
-  %meta{ property: 'og:site_name',   content: site_name }
-  %meta{ property: 'og:title',       content: page_title }
+  %meta{ property: 'og:type', content: "object" }
+  %meta{ property: 'og:site_name', content: site_name }
+  %meta{ property: 'og:title', content: page_title }
   %meta{ property: 'og:description', content: page_description }
-  %meta{ property: 'og:image',       content: page_image }
+  %meta{ property: 'og:image', content: page_image }
   %meta{ property: 'og:image:width', content: '64' }
   %meta{ property: 'og:image:height', content: '64' }
-  %meta{ property: 'og:url',         content: request.base_url + request.fullpath }
+  %meta{ property: 'og:url', content: request.base_url + request.fullpath }
 
   -# Twitter Card - https://dev.twitter.com/cards/types/summary
-  %meta{ property: 'twitter:card',         content: "summary" }
-  %meta{ property: 'twitter:title',        content: page_title }
-  %meta{ property: 'twitter:description',  content: page_description }
-  %meta{ property: 'twitter:image',        content: page_image }
+  %meta{ property: 'twitter:card', content: "summary" }
+  %meta{ property: 'twitter:title', content: page_title }
+  %meta{ property: 'twitter:description', content: page_description }
+  %meta{ property: 'twitter:image', content: page_image }
   = page_card_meta_tags
 
   %title= page_title(site_name)
@@ -33,8 +33,8 @@
   = favicon_link_tag favicon, id: 'favicon', data: { original_href: favicon }, type: 'image/png'
 
   = stylesheet_link_tag "application", media: "all"
-  = stylesheet_link_tag "print",       media: "print"
-  = stylesheet_link_tag "test",        media: "all" if Rails.env.test?
+  = stylesheet_link_tag "print", media: "print"
+  = stylesheet_link_tag "test", media: "all" if Rails.env.test?
   = stylesheet_link_tag 'performance_bar' if performance_bar_enabled?
   = stylesheet_link_tag 'csslab' if Feature.enabled?(:csslab)
 
@@ -61,10 +61,10 @@
   %meta{ name: 'theme-color', content: '#474D57' }
 
   -# Apple Safari/iOS home screen icons
-  = favicon_link_tag 'touch-icon-iphone.png',        rel: 'apple-touch-icon'
-  = favicon_link_tag 'touch-icon-ipad.png',          rel: 'apple-touch-icon', sizes: '76x76'
+  = favicon_link_tag 'touch-icon-iphone.png', rel: 'apple-touch-icon'
+  = favicon_link_tag 'touch-icon-ipad.png', rel: 'apple-touch-icon', sizes: '76x76'
   = favicon_link_tag 'touch-icon-iphone-retina.png', rel: 'apple-touch-icon', sizes: '120x120'
-  = favicon_link_tag 'touch-icon-ipad-retina.png',   rel: 'apple-touch-icon', sizes: '152x152'
+  = favicon_link_tag 'touch-icon-ipad-retina.png', rel: 'apple-touch-icon', sizes: '152x152'
   %link{ rel: 'mask-icon', href: image_path('logo.svg'), color: 'rgb(226, 67, 41)' }
 
   -# Windows 8 pinned site tile

app/views/layouts/application.html.haml

@@ -6,7 +6,7 @@
     = render "layouts/init_client_detection_flags"
     = render 'peek/bar'
     = header_message
-    = render  partial: "layouts/header/default", locals: { project: @project, group: @group }
+    = render partial: "layouts/header/default", locals: { project: @project, group: @group }
     = render 'layouts/page', sidebar: sidebar, nav: nav
     = footer_message
 

app/views/layouts/group.html.haml

 - page_title       @group.name
 - page_description @group.description  unless page_description
 - header_title     group_title(@group) unless header_title
-- nav            "group"
+- nav "group"
 - @left_sidebar = true
 
 - content_for :page_specific_javascripts do

app/views/layouts/header/_new_dropdown.haml

-%li.header-new.dropdown{ data: { track_label: "new_dropdown", track_event: "click_dropdown"  } }
+%li.header-new.dropdown{ data: { track_label: "new_dropdown", track_event: "click_dropdown" } }
   = link_to new_project_path, class: "header-new-dropdown-toggle has-tooltip qa-new-menu-toggle", title: _("New..."), ref: 'tooltip', aria: { label: _("New...") }, data: { toggle: 'dropdown', placement: 'bottom', container: 'body', display: 'static' } do
     = sprite_icon('plus-square', size: 16)
     = sprite_icon('angle-down', css_class: 'caret-down')

app/views/layouts/nav/_dashboard.html.haml

@@ -10,7 +10,7 @@
         = render "layouts/nav/projects_dropdown/show"
 
   - if dashboard_nav_link?(:groups)
-    = nav_link(controller: ['dashboard/groups', 'explore/groups'], html_options: { id: 'nav-groups-dropdown', class: "home dropdown header-groups qa-groups-dropdown", data: { track_label: "groups_dropdown", track_event: "click_dropdown"  } }) do
+    = nav_link(controller: ['dashboard/groups', 'explore/groups'], html_options: { id: 'nav-groups-dropdown', class: "home dropdown header-groups qa-groups-dropdown", data: { track_label: "groups_dropdown", track_event: "click_dropdown" } }) do
       %button{ type: 'button', data: { toggle: "dropdown" } }
         = _('Groups')
         = sprite_icon('angle-down', css_class: 'caret-down')

app/views/layouts/nav/sidebar/_project.html.haml

@@ -170,7 +170,7 @@
               = _('CI / CD')
 
           %ul.sidebar-sub-level-items
-            = nav_link(controller: [:pipelines, :builds, :jobs, :pipeline_schedules, :artifacts], html_options: { class: "fly-out-top-item" }  ) do
+            = nav_link(controller: [:pipelines, :builds, :jobs, :pipeline_schedules, :artifacts], html_options: { class: "fly-out-top-item" }) do
               = link_to project_pipelines_path(@project) do
                 %strong.fly-out-top-item-name
                   =  _('CI / CD')

app/views/layouts/snippets.html.haml

-- header_title  _("Snippets"), snippets_path
+- header_title _("Snippets"), snippets_path
 
 - content_for :page_specific_javascripts do
   - if @snippet && current_user

app/views/projects/_export.html.haml

@@ -33,9 +33,9 @@
     %p
       Once the exported file is ready, you will receive a notification email with a download link, or you can download it from this page.
     - if project.export_status == :finished
-      = link_to 'Download export',  download_export_project_path(project),
+      = link_to 'Download export', download_export_project_path(project),
               rel: 'nofollow', download: '', method: :get, class: "btn btn-default"
-      = link_to 'Generate new export',  generate_new_export_project_path(project),
+      = link_to 'Generate new export', generate_new_export_project_path(project),
               method: :post, class: "btn btn-default"
     - else
       = link_to 'Export project', export_project_path(project),

app/views/projects/_home_panel.html.haml

@@ -23,7 +23,7 @@
               = sprite_icon('tag', size: 16, css_class: 'icon append-right-4')
               = @project.topics_to_show
               - if @project.has_extra_topics?
-                = _("+ %{count} more") % { count: @project.count_of_extra_topics_not_shown  }
+                = _("+ %{count} more") % { count: @project.count_of_extra_topics_not_shown }
 
 
     .project-repo-buttons.col-md-12.col-lg-6.d-inline-flex.flex-wrap.justify-content-lg-end

app/views/projects/_issuable_by_email.html.haml

@@ -21,7 +21,7 @@
             = clipboard_button(target: '#issuable_email', class: 'btn btn-clipboard input-group-text btn-transparent d-none d-sm-block')
 
             - if issuable_type == 'issue'
-              - enter_title_text =  _('Enter the issue title')
+              - enter_title_text = _('Enter the issue title')
               - enter_description_text = _('Enter the issue description')
             - else
               - enter_title_text = _('Enter the merge request title')

app/views/projects/_new_project_fields.html.haml

@@ -43,7 +43,7 @@
   = f.label :description, class: 'label-bold' do
     Project description
     %span (optional)
-  = f.text_area :description, placeholder: 'Description format',  class: "form-control", rows: 3, maxlength: 250, data: { track_label: "#{track_label}", track_event: "activate_form_input", track_property: "project_description", track_value: "" }
+  = f.text_area :description, placeholder: 'Description format', class: "form-control", rows: 3, maxlength: 250, data: { track_label: "#{track_label}", track_event: "activate_form_input", track_property: "project_description", track_value: "" }
 
 = f.label :visibility_level, class: 'label-bold' do
   Visibility Level

app/views/projects/buttons/_dropdown.html.haml

@@ -35,8 +35,8 @@
       - elsif can_collaborate_with_project?(@project)
         %li= link_to _('New file'), project_new_blob_path(@project, @project.default_branch || 'master')
       - elsif create_mr_from_new_fork
-        - continue_params = { to:         project_new_blob_path(@project, @project.default_branch || 'master'),
-                              notice:     edit_in_new_fork_notice,
+        - continue_params = { to: project_new_blob_path(@project, @project.default_branch || 'master'),
+                              notice: edit_in_new_fork_notice,
                               notice_now: edit_in_new_fork_notice_now }
         - fork_path = project_forks_path(@project, namespace_key: current_user.namespace.id, continue: continue_params)
         %li= link_to _('New file'), fork_path, method: :post

app/views/projects/commit/_commit_box.html.haml

@@ -49,7 +49,7 @@
           #{ _('Download') }
         - unless @commit.parents.length > 1
           %li= link_to s_("DownloadCommit|Email Patches"), project_commit_path(@project, @commit, format: :patch), class: "qa-email-patches"
-        %li= link_to s_("DownloadCommit|Plain Diff"),    project_commit_path(@project, @commit, format: :diff), class: "qa-plain-diff"
+        %li= link_to s_("DownloadCommit|Plain Diff"), project_commit_path(@project, @commit, format: :diff), class: "qa-plain-diff"
 
 .commit-box{ data: { project_path: project_path(@project) } }
   %h3.commit-title

app/views/projects/issues/_merge_requests.html.haml

@@ -19,7 +19,7 @@
                 = render partial: 'projects/issues/merge_requests_status', locals: { merge_request: merge_request, css_class: 'd-none d-xl-block append-right-8' }
                 = link_to merge_request.title, merge_request_path(merge_request), { class: 'mr-title-link'}
               .item-meta
-                = render partial: 'projects/issues/merge_requests_status', locals: { merge_request: merge_request,  css_class: 'd-xl-none d-lg-block append-right-5' }
+                = render partial: 'projects/issues/merge_requests_status', locals: { merge_request: merge_request, css_class: 'd-xl-none d-lg-block append-right-5' }
                 %span.d-flex.align-items-center.append-right-8.mr-item-path.item-path-id.mt-0
                   %span.path-id-text.bold.text-truncate{ data: { toggle: 'tooltip'}, title: merge_request.target_project.full_path }
                     = merge_request.target_project.full_path

app/views/projects/issues/_merge_requests_status.html.haml

@@ -18,5 +18,5 @@
 
 - mr_status_tooltip = "<div><span class=\"bold\">#{mr_status_title}</span> #{time_ago_in_words(mr_status_date)} ago</div><span class=\"text-tertiary\">#{l(mr_status_date.to_time, format: time_format)}</span>"
 
-%span.mr-status-wrapper.suggestion-help-hover{ class: css_class, data: { toggle: 'tooltip', placement: 'bottom', html: 'true', title:  mr_status_tooltip } }
+%span.mr-status-wrapper.suggestion-help-hover{ class: css_class, data: { toggle: 'tooltip', placement: 'bottom', html: 'true', title: mr_status_tooltip } }
   = sprite_icon(mr_status_icon, size: 16, css_class: "merge-request-status #{mr_status_class}")

app/views/projects/issues/_new_branch.html.haml

@@ -30,7 +30,7 @@
                 = icon('check', class: 'icon')
                 = _('Create merge request and branch')
 
-          %li{ class: [!can_create_merge_request && 'droplab-item-selected'], role: 'button', data: {  value: 'create-branch', text: _('Create branch') } }
+          %li{ class: [!can_create_merge_request && 'droplab-item-selected'], role: 'button', data: { value: 'create-branch', text: _('Create branch') } }
             .menu-item
               = icon('check', class: 'icon')
               = _('Create branch')

app/views/projects/milestones/_form.html.haml

 = form_for [@project.namespace.becomes(Namespace), @project, @milestone],
-           html: {class: 'milestone-form common-note-form js-quick-submit js-requires-input'},
-           data: { markdown_version: @milestone.cached_markdown_version }  do |f|
+           html: { class: 'milestone-form common-note-form js-quick-submit js-requires-input' },
+           data: { markdown_version: @milestone.cached_markdown_version } do |f|
   = form_errors(@milestone)
   .row
     .col-md-6

app/views/projects/pipeline_schedules/_form.html.haml

@@ -34,7 +34,7 @@
             = n_('Reveal value', 'Reveal values', @schedule.variables.size)
   .form-group.row
     .col-md-9
-      = f.label  :active, s_('PipelineSchedules|Activated'), class: 'label-bold'
+      = f.label :active, s_('PipelineSchedules|Activated'), class: 'label-bold'
       %div
         = f.check_box :active, required: false, value: @schedule.active?
         = _('Active')

app/views/projects/project_members/_team.html.haml

@@ -6,7 +6,7 @@
     %span.flex-project-title
       = _("Members of <strong>%{project_name}</strong>").html_safe % { project_name: sanitize_project_name(project.name) }
     %span.badge.badge-pill= members.total_count
-    = form_tag project_project_members_path(project), method: :get, class: 'form-inline member-search-form flex-project-members-form'  do
+    = form_tag project_project_members_path(project), method: :get, class: 'form-inline member-search-form flex-project-members-form' do
       .form-group
         .position-relative
           = search_field_tag :search, params[:search], { placeholder: _('Find existing members by name'), class: 'form-control', spellcheck: false }

app/views/projects/project_templates/_built_in_templates.html.haml

@@ -12,6 +12,6 @@
       %a.btn.btn-default.append-right-10{ href: template.preview, rel: 'noopener noreferrer', target: '_blank', data: { track_label: "create_from_template", track_property: "template_preview", track_event: "click_button", track_value: template.name } }
         = _("Preview")
       %label.btn.btn-success.template-button.choose-template.append-bottom-0{ for: template.name }
-        %input{ type: "radio",  autocomplete: "off", name: "project[template_name]", id: template.name, value: template.name, data: { track_label: "create_from_template", track_property: "template_use", track_event: "click_button" } }
+        %input{ type: "radio", autocomplete: "off", name: "project[template_name]", id: template.name, value: template.name, data: { track_label: "create_from_template", track_property: "template_use", track_event: "click_button" } }
         %span
           = _("Use template")

app/views/projects/serverless/functions/index.html.haml

@@ -5,7 +5,7 @@
 - status_path = project_serverless_functions_path(@project, format: :json)
 - clusters_path = project_clusters_path(@project)
 
-.serverless-functions-page.js-serverless-functions-page{ data: { status_path: status_path, installed: @installed,  clusters_path: clusters_path, help_path: help_page_path('user/project/clusters/serverless/index') } }
+.serverless-functions-page.js-serverless-functions-page{ data: { status_path: status_path, installed: @installed, clusters_path: clusters_path, help_path: help_page_path('user/project/clusters/serverless/index') } }
 
 %div{ class: [container_class, ('limit-container-width' unless fluid_layout)] }
   .js-serverless-functions-notice

app/views/projects/tree/_tree_header.html.haml

@@ -40,27 +40,24 @@
                     #{ _('New directory') }
               - elsif can?(current_user, :fork_project, @project) && can?(current_user, :create_merge_request_in, @project)
                 %li
-                  - continue_params = { to:         project_new_blob_path(@project, @id),
-                                        notice:     edit_in_new_fork_notice,
+                  - continue_params = { to: project_new_blob_path(@project, @id),
+                                        notice: edit_in_new_fork_notice,
                                         notice_now: edit_in_new_fork_notice_now }
-                  - fork_path = project_forks_path(@project, namespace_key:  current_user.namespace.id,
-                                                                                          continue:       continue_params)
+                  - fork_path = project_forks_path(@project, namespace_key: current_user.namespace.id, continue: continue_params)
                   = link_to fork_path, method: :post do
                     #{ _('New file') }
                 %li
-                  - continue_params = { to:         request.fullpath,
-                                        notice:     edit_in_new_fork_notice + " Try to upload a file again.",
+                  - continue_params = { to: request.fullpath,
+                                        notice: edit_in_new_fork_notice + " Try to upload a file again.",
                                         notice_now: edit_in_new_fork_notice_now }
-                  - fork_path = project_forks_path(@project, namespace_key:  current_user.namespace.id,
-                                                                                          continue:       continue_params)
+                  - fork_path = project_forks_path(@project, namespace_key: current_user.namespace.id, continue: continue_params)
                   = link_to fork_path, method: :post do
                     #{ _('Upload file') }
                 %li
-                  - continue_params = { to:         request.fullpath,
-                                        notice:     edit_in_new_fork_notice + " Try to create a new directory again.",
+                  - continue_params = { to: request.fullpath,
+                                        notice: edit_in_new_fork_notice + " Try to create a new directory again.",
                                         notice_now: edit_in_new_fork_notice_now }
-                  - fork_path = project_forks_path(@project, namespace_key:  current_user.namespace.id,
-                                                                                          continue:       continue_params)
+                  - fork_path = project_forks_path(@project, namespace_key: current_user.namespace.id, continue: continue_params)
                   = link_to fork_path, method: :post do
                     #{ _('New directory') }
 

app/views/shared/boards/components/_sidebar.html.haml

-%board-sidebar{ "inline-template" => true,  ":current-user" => (UserSerializer.new.represent(current_user) || {}).to_json }
+%board-sidebar{ "inline-template" => true, ":current-user" => (UserSerializer.new.represent(current_user) || {}).to_json }
   %transition{ name: "boards-sidebar-slide" }
     %aside.right-sidebar.right-sidebar-expanded.issue-boards-sidebar{ "v-show" => "showSidebar" }
       .issuable-sidebar

app/views/shared/issuable/_bulk_update_sidebar.html.haml

@@ -2,7 +2,7 @@
 
 %aside.issues-bulk-update.js-right-sidebar.right-sidebar{ "aria-live" => "polite", data: { 'signed-in': current_user.present? } }
   .issuable-sidebar.hidden
-    = form_tag [:bulk_update, @project.namespace.becomes(Namespace), @project, type], method: :post, class: "bulk-update"  do
+    = form_tag [:bulk_update, @project.namespace.becomes(Namespace), @project, type], method: :post, class: "bulk-update" do
       .block.issuable-sidebar-header
         .filter-item.inline.update-issues-btn.float-left
           = button_tag "Update all", class: "btn update-selected-issues btn-info", disabled: true
@@ -50,4 +50,3 @@
 
       = hidden_field_tag "update[issuable_ids]", []
       = hidden_field_tag :state_event, params[:state_event]
-

app/workers/namespaceless_project_destroy_worker.rb

@@ -17,7 +17,7 @@ class NamespacelessProjectDestroyWorker
       return
     end
 
-    return if project.namespace  # Reject doing anything for projects that *do* have a namespace
+    return if project.namespace # Reject doing anything for projects that *do* have a namespace
 
     project.team.truncate
 

config.ru

@@ -19,7 +19,7 @@ if defined?(Unicorn)
   Unicorn::StreamInput.send(:public, :eof?) # rubocop:disable GitlabSecurity/PublicSend
 end
 
-require ::File.expand_path('../config/environment',  __FILE__)
+require ::File.expand_path('../config/environment', __FILE__)
 
 warmup do |app|
   client = Rack::MockRequest.new(app)

config/initializers/1_settings.rb

@@ -7,7 +7,7 @@ Settings.ldap['enabled'] = false if Settings.ldap['enabled'].nil?
 Settings.ldap['sync_time'] = 3600 if Settings.ldap['sync_time'].nil?
 Settings.ldap['schedule_sync_daily'] = 1 if Settings.ldap['schedule_sync_daily'].nil?
 Settings.ldap['schedule_sync_hour'] = 1 if Settings.ldap['schedule_sync_hour'].nil?
-Settings.ldap['schedule_sync_minute'] = 30  if Settings.ldap['schedule_sync_minute'].nil?
+Settings.ldap['schedule_sync_minute'] = 30 if Settings.ldap['schedule_sync_minute'].nil?
 
 # backwards compatibility, we only have one host
 if Settings.ldap['enabled'] || Rails.env.test?

config/initializers/8_devise.rb

@@ -178,7 +178,7 @@ Devise.setup do |config|
 
   # Configure the default scope given to Warden. By default it's the first
   # devise role declared in your routes (usually :user).
-  config.default_scope = :user  # now have an :email scope as well, so set the default
+  config.default_scope = :user # now have an :email scope as well, so set the default
 
   # Configure sign_out behavior.
   # Sign_out action can be scoped (i.e. /users/sign_out affects only :user scope).

config/initializers/doorkeeper_openid_connect.rb

@@ -31,7 +31,7 @@ Doorkeeper::OpenidConnect.configure do
 
       o.claim(:name)           { |user| user.name }
       o.claim(:nickname)       { |user| user.username }
-      o.claim(:email)          { |user| user.public_email  }
+      o.claim(:email)          { |user| user.public_email }
       o.claim(:email_verified) { |user| true if user.public_email? }
       o.claim(:website)        { |user| user.full_website_url if user.website_url? }
       o.claim(:profile)        { |user| Gitlab::Routing.url_helpers.user_url user }

config/initializers/postgresql_cte.rb

@@ -108,7 +108,7 @@ module ActiveRecord
         when String
           with_value
         when Hash
-          with_value.map  do |name, expression|
+          with_value.map do |name, expression|
             case expression
             when String
               select = Arel::Nodes::SqlLiteral.new "(#{expression})"

config/routes/repository.rb

@@ -62,7 +62,7 @@ scope format: false do
     resources :protected_tags, only: [:index, :show, :create, :update, :destroy]
   end
 
-  scope constraints: { id: /.+/ }  do
+  scope constraints: { id: /.+/ } do
     scope controller: :blob do
       get '/new/*id', action: :new, as: :new_blob
       post '/create/*id', action: :create, as: :create_blob

db/migrate/20170927095921_add_ci_builds_index_for_jobscontroller.rb

@@ -28,7 +28,7 @@ class AddCiBuildsIndexForJobscontroller < ActiveRecord::Migration[4.2]
   disable_ddl_transaction!
 
   def up
-    add_concurrent_index :ci_builds, [:project_id, :id]  unless index_exists? :ci_builds, [:project_id, :id]
+    add_concurrent_index :ci_builds, [:project_id, :id] unless index_exists? :ci_builds, [:project_id, :id]
     remove_concurrent_index :ci_builds, :project_id if index_exists? :ci_builds, :project_id
   end
 

db/post_migrate/20170317162059_update_upload_paths_to_system.rb

@@ -48,7 +48,7 @@ class UpdateUploadPathsToSystem < ActiveRecord::Migration[4.2]
   end
 
   def new_upload_dir
-    File.join(base_directory, "-",  "system")
+    File.join(base_directory, "-", "system")
   end
 
   def arel_table

ee/app/helpers/ee/gitlab_routing_helper.rb

@@ -49,7 +49,7 @@ module EE
 
     url_helper :user_group_saml_omniauth_metadata
     def user_group_saml_omniauth_metadata_path(group)
-      params = { group_path: group.path, token:  group.saml_discovery_token }
+      params = { group_path: group.path, token: group.saml_discovery_token }
       path = '/users/auth/group_saml/metadata'
 
       ActionDispatch::Http::URL.path_for(path: path, params: params)

ee/app/models/approver.rb

 # frozen_string_literal: true
 
 class Approver < ActiveRecord::Base
-  belongs_to :target, polymorphic: true  # rubocop:disable Cop/PolymorphicAssociations
+  belongs_to :target, polymorphic: true # rubocop:disable Cop/PolymorphicAssociations
   belongs_to :user
 
   include ApproverMigrateHook

ee/app/models/approver_group.rb

 # frozen_string_literal: true
 
 class ApproverGroup < ActiveRecord::Base
-  belongs_to :target, polymorphic: true  # rubocop:disable Cop/PolymorphicAssociations
+  belongs_to :target, polymorphic: true # rubocop:disable Cop/PolymorphicAssociations
   belongs_to :group
 
   include ApproverMigrateHook

ee/app/models/draft_note.rb

@@ -59,7 +59,7 @@ class DraftNote < ActiveRecord::Base
   end
 
   def type
-    return 'DiffNote' if  on_diff?
+    return 'DiffNote' if on_diff?
     return 'DiscussionNote' if discussion_id.present?
 
     'Note'

ee/app/models/ee/appearance.rb

@@ -9,10 +9,10 @@ module EE
       cache_markdown_field :footer_message, pipeline: :broadcast_message
 
       validates :message_background_color, allow_blank: true, color: true
-      validates :message_font_color,  allow_blank: true, color: true
+      validates :message_font_color, allow_blank: true, color: true
 
       default_value_for :message_background_color, '#E75E40'
-      default_value_for :message_font_color,  '#FFFFFF'
+      default_value_for :message_font_color, '#FFFFFF'
     end
 
     def show_header?

ee/app/models/ee/epic.rb

@@ -248,7 +248,7 @@ module EE
     def to_reference(from = nil, full: false)
       reference = "#{self.class.reference_prefix}#{iid}"
 
-      return reference unless  (cross_reference?(from) && !group.projects.include?(from)) || full
+      return reference unless (cross_reference?(from) && !group.projects.include?(from)) || full
 
       "#{group.full_path}#{reference}"
     end

ee/app/models/ee/user.rb

@@ -31,7 +31,7 @@ module EE
       has_many :epics,                    foreign_key: :author_id
       has_many :assigned_epics,           foreign_key: :assignee_id, class_name: "Epic"
       has_many :path_locks,               dependent: :destroy # rubocop: disable Cop/ActiveRecordDependent
-      has_many :vulnerability_feedback,  foreign_key: :author_id, class_name: 'Vulnerabilities::Feedback'
+      has_many :vulnerability_feedback, foreign_key: :author_id, class_name: 'Vulnerabilities::Feedback'
 
       has_many :approvals,                dependent: :destroy # rubocop: disable Cop/ActiveRecordDependent
       has_many :approvers,                dependent: :destroy # rubocop: disable Cop/ActiveRecordDependent

ee/app/models/geo/project_registry.rb

@@ -285,7 +285,7 @@ class Geo::ProjectRegistry < Geo::BaseRegistry
   # @see REGISTRY_TYPES
   def should_be_redownloaded?(type)
     ensure_valid_type!(type)
-    return true if public_send("force_to_redownload_#{type}")  # rubocop:disable GitlabSecurity/PublicSend
+    return true if public_send("force_to_redownload_#{type}") # rubocop:disable GitlabSecurity/PublicSend
 
     retry_count(type) > RETRIES_BEFORE_REDOWNLOAD
   end

ee/app/models/geo_node.rb

@@ -127,7 +127,7 @@ class GeoNode < ActiveRecord::Base
   end
 
   def url=(value)
-    value += '/'  if value.present? && !value.end_with?('/')
+    value += '/' if value.present? && !value.end_with?('/')
 
     write_attribute(:url, value)
 
@@ -280,7 +280,7 @@ class GeoNode < ActiveRecord::Base
       errors.add(:url, 'scheme must be http or https')
     end
   rescue URI::InvalidURIError
-    errors.add(:url,  'is invalid')
+    errors.add(:url, 'is invalid')
   end
 
   def update_clone_url

ee/app/services/ee/ci/register_job_service.rb

@@ -31,7 +31,7 @@ module EE
 
         # select projects which have allowed number of shared runner minutes or are public
         super
-          .where("projects.visibility_level=? OR (#{builds_check_limit.to_sql})=1",  # rubocop:disable GitlabSecurity/SqlInjection
+          .where("projects.visibility_level=? OR (#{builds_check_limit.to_sql})=1", # rubocop:disable GitlabSecurity/SqlInjection
                 ::Gitlab::VisibilityLevel::PUBLIC)
       end
       # rubocop: enable CodeReuse/ActiveRecord

ee/app/views/groups/analytics/show.html.haml

@@ -39,7 +39,7 @@
   - mr_created_count = @events.merge_requests.created.count
   - mr_merged_count = @events.merge_requests.merged.count
   - if mr_created_count > 0 || mr_merged_count > 0
-    = _('<strong>%{created_count}</strong> created, <strong>%{accepted_count}</strong> accepted.').html_safe % { created_count: mr_created_count, accepted_count: mr_merged_count  }
+    = _('<strong>%{created_count}</strong> created, <strong>%{accepted_count}</strong> accepted.').html_safe % { created_count: mr_created_count, accepted_count: mr_merged_count }
   - else
     = _('No merge requests for the selected time period.')
 

ee/app/views/groups/security/dashboard/show.html.haml

 - breadcrumb_title _("Security Dashboard")
-- page_title  _("Security Dashboard")
+- page_title _("Security Dashboard")
 
 #js-group-security-dashboard{ data: { vulnerabilities_endpoint: group_security_vulnerabilities_path(@group),
   vulnerabilities_summary_endpoint: summary_group_security_vulnerabilities_path(@group),

ee/app/views/projects/feature_flags/index.html.haml

@@ -6,4 +6,4 @@
   "error-state-svg-path" => image_path('illustrations/feature_flag.svg'),
   "feature-flags-help-page-path" => help_page_path("user/project/operations/feature_flags"),
   "can-user-admin-feature-flag" => can?(current_user, :admin_feature_flag, @project),
-  "new-feature-flag-path" => can?(current_user, :create_feature_flag, @project) ?  new_project_feature_flag_path(@project): nil } }
+  "new-feature-flag-path" => can?(current_user, :create_feature_flag, @project) ? new_project_feature_flag_path(@project): nil } }

ee/app/views/users/_custom_project_templates.html.haml

@@ -14,7 +14,7 @@
           %a.btn.btn-default.append-right-10{ href: project_path(template), rel: 'noopener noreferrer', target: '_blank' }
             = _('Preview')
           %label.btn.btn-success.custom-template-button.choose-template.append-bottom-0{ for: template.name }
-            %input{ type: "radio",  autocomplete: "off", name: "project[template_name]", id: template.name, value: template.name }
+            %input{ type: "radio", autocomplete: "off", name: "project[template_name]", id: template.name, value: template.name }
             %span
               = _('Use template')
 

ee/app/views/users/_custom_project_templates_from_groups.html.haml

@@ -33,7 +33,7 @@
               %a.btn.btn-default.append-right-10{ href: project_path(project), rel: 'noopener noreferrer', target: '_blank' }
                 = _('Preview')
               %label.btn.btn-success.custom-template-button.choose-template.append-bottom-0{ for: project.name }
-                %input{ type: "radio",  autocomplete: "off", name: "project[template_name]", id: project.name, value: project.name, data: { subgroup_id: project.namespace_id } }
+                %input{ type: "radio", autocomplete: "off", name: "project[template_name]", id: project.name, value: project.name, data: { subgroup_id: project.namespace_id } }
                 %span.qa-use-template-button
                   = _('Use template')
 

ee/db/migrate/20180626171125_add_feature_flags_to_projects.rb

@@ -20,7 +20,7 @@ class AddFeatureFlagsToProjects < ActiveRecord::Migration[4.2]
       t.index [:project_id, :name], unique: true
     end
 
-    create_table :operations_feature_flags_clients, id: :bigserial  do |t|
+    create_table :operations_feature_flags_clients, id: :bigserial do |t|
       t.integer :project_id, null: false
       t.string :token, null: false
 

ee/db/migrate/20180917171533_create_vulnerability_occurrences.rb

@@ -6,7 +6,7 @@ class CreateVulnerabilityOccurrences < ActiveRecord::Migration[4.2]
   DOWNTIME = false
 
   def change
-    create_table :vulnerability_occurrences, id: :bigserial  do |t|
+    create_table :vulnerability_occurrences, id: :bigserial do |t|
       t.timestamps_with_timezone null: false
 
       t.integer :severity, null: false, limit: 2

ee/db/migrate/20180930171532_recreate_vulnerability_occurrences_and_vulnerability_occurrence_identifiers.rb

@@ -9,7 +9,7 @@ class RecreateVulnerabilityOccurrencesAndVulnerabilityOccurrenceIdentifiers < Ac
     drop_table :vulnerability_occurrence_identifiers
     drop_table :vulnerability_occurrences
 
-    create_table :vulnerability_occurrences, id: :bigserial  do |t|
+    create_table :vulnerability_occurrences, id: :bigserial do |t|
       t.timestamps_with_timezone null: false
 
       t.integer :severity, null: false, limit: 2
@@ -62,7 +62,7 @@ class RecreateVulnerabilityOccurrencesAndVulnerabilityOccurrenceIdentifiers < Ac
     drop_table :vulnerability_occurrence_identifiers
     drop_table :vulnerability_occurrences
 
-    create_table :vulnerability_occurrences, id: :bigserial  do |t|
+    create_table :vulnerability_occurrences, id: :bigserial do |t|
       t.timestamps_with_timezone null: false
 
       t.integer :severity, null: false, limit: 2

ee/db/migrate/20181004131020_change_vuln_occurrence_columns.rb

@@ -7,7 +7,7 @@ class ChangeVulnOccurrenceColumns < ActiveRecord::Migration[4.2]
     drop_table :vulnerability_occurrence_identifiers
     drop_table :vulnerability_occurrences
 
-    create_table :vulnerability_occurrences, id: :bigserial  do |t|
+    create_table :vulnerability_occurrences, id: :bigserial do |t|
       t.timestamps_with_timezone null: false
 
       t.integer :severity, null: false, limit: 2
@@ -59,7 +59,7 @@ class ChangeVulnOccurrenceColumns < ActiveRecord::Migration[4.2]
     drop_table :vulnerability_occurrence_identifiers
     drop_table :vulnerability_occurrences
 
-    create_table :vulnerability_occurrences, id: :bigserial  do |t|
+    create_table :vulnerability_occurrences, id: :bigserial do |t|
       t.timestamps_with_timezone null: false
 
       t.integer :severity, null: false, limit: 2

ee/lib/api/managed_licenses.rb

@@ -27,7 +27,7 @@ module API
       requires :id, type: String, desc: 'The ID of a project'
     end
 
-    resource :projects, requirements: API::NAMESPACE_OR_PROJECT_REQUIREMENTS  do
+    resource :projects, requirements: API::NAMESPACE_OR_PROJECT_REQUIREMENTS do
       desc 'Get project software license policies' do
         success Entities::ManagedLicense
       end

ee/lib/ee/api/entities.rb

@@ -350,7 +350,7 @@ module EE
           number_to_percentage(node.repositories_synced_in_percentage, precision: 2)
         end
 
-        expose :wikis_count  # Deprecated
+        expose :wikis_count # Deprecated
         expose :wikis_failed_count
         expose :wikis_synced_count
         expose :wikis_synced_in_percentage do |node|

ee/lib/ee/gitlab/usage_data.rb

@@ -84,7 +84,7 @@ module EE
         }
 
         results = count(::Ci::Build.where(name: types.keys).group(:name), fallback: Hash.new(-1))
-        results.each_with_object({}) { |(key, value), response| response[types[key.to_sym]] = value  }
+        results.each_with_object({}) { |(key, value), response| response[types[key.to_sym]] = value }
       end
       # rubocop: enable CodeReuse/ActiveRecord
 

ee/lib/tasks/gitlab/elastic.rake

@@ -31,7 +31,7 @@ namespace :gitlab do
     end
 
     desc "GitLab | Elasticsearch | Index project repositories"
-    task index_repositories: :environment  do
+    task index_repositories: :environment do
       print "Indexing project repositories..."
 
       Sidekiq::Logging.logger = Logger.new(STDOUT)
@@ -41,7 +41,7 @@ namespace :gitlab do
     end
 
     desc "GitLab | Elasticsearch | Index wiki repositories"
-    task index_wikis: :environment  do
+    task index_wikis: :environment do
       projects = apply_project_filters(Project.with_wiki_enabled)
 
       projects.find_each do |project|

ee/spec/controllers/groups/epics_controller_spec.rb

@@ -278,7 +278,7 @@ describe Groups::EpicsController do
         group.add_developer(user)
       end
 
-      context 'with correct basic params'  do
+      context 'with correct basic params' do
         it 'returns status 200' do
           update_epic(epic, params)
 

ee/spec/controllers/groups/security/vulnerabilities_controller_spec.rb

@@ -246,7 +246,7 @@ describe Groups::Security::VulnerabilitiesController do
   end
 
   describe 'GET history.json' do
-    subject { get :history,  params: { group_id: group }, format: :json }
+    subject { get :history, params: { group_id: group }, format: :json }
 
     context 'when security dashboard feature is disabled' do
       before do
@@ -350,7 +350,7 @@ describe Groups::Security::VulnerabilitiesController do
 
         it 'returns filtered history if filters are enabled' do
           travel_to(Time.zone.parse('2019-02-10')) do
-            get :history,  params: { group_id: group, report_type: %w[dependency_scanning sast] }, format: :json
+            get :history, params: { group_id: group, report_type: %w[dependency_scanning sast] }, format: :json
           end
 
           expect(response).to have_gitlab_http_status(200)

ee/spec/controllers/projects/issues_controller_spec.rb

@@ -251,7 +251,7 @@ describe Projects::IssuesController do
   end
 
   describe 'GET #discussions' do
-    let(:issue)   { create(:issue, project: project) }
+    let(:issue) { create(:issue, project: project) }
     let!(:discussion) { create(:discussion_note_on_issue, noteable: issue, project: issue.project) }
 
     context 'with a related system note' do

ee/spec/controllers/unsubscribes_controller_spec.rb

@@ -5,7 +5,7 @@ describe UnsubscribesController do
 
   describe "show" do
     it "responds with success" do
-      get :show, params: { email:  Base64.urlsafe_encode64('me@example.com') }
+      get :show, params: { email: Base64.urlsafe_encode64('me@example.com') }
 
       assert_response :success
     end

ee/spec/factories/geo/project_registry.rb

@@ -46,7 +46,7 @@ FactoryBot.define do
     trait :repository_sync_failed do
       sync_failed
 
-      last_wiki_successful_sync_at  { 5.days.ago }
+      last_wiki_successful_sync_at { 5.days.ago }
       resync_wiki false
       wiki_retry_count nil
     end

ee/spec/factories/vulnerabilities/occurrences.rb

@@ -8,7 +8,7 @@ FactoryBot.define do
   factory :vulnerabilities_occurrence, class: Vulnerabilities::Occurrence do
     name 'Cipher with no integrity'
     project
-    sequence(:uuid) { generate(:vulnerability_occurrence_uuid)  }
+    sequence(:uuid) { generate(:vulnerability_occurrence_uuid) }
     project_fingerprint { generate(:project_fingerprint) }
     primary_identifier factory: :vulnerabilities_identifier
     location_fingerprint '4e5b6966dd100170b4b1ad599c7058cce91b57b4'

ee/spec/features/epics/todo_spec.rb

@@ -2,8 +2,8 @@ require 'rails_helper'
 
 describe 'Manually create a todo item from epic', :js do
   let(:group) { create(:group) }
-  let(:epic)   { create(:epic, group: group) }
-  let(:user)    { create(:user)}
+  let(:epic) { create(:epic, group: group) }
+  let(:user) { create(:user)}
 
   before do
     stub_licensed_features(epics: true)

ee/spec/features/groups/group_settings_spec.rb

@@ -3,8 +3,8 @@ require 'spec_helper'
 describe 'Edit group settings' do
   include Select2Helper
 
-  let(:user)  { create(:user) }
-  let(:developer)  { create(:user) }
+  let(:user) { create(:user) }
+  let(:developer) { create(:user) }
   let(:group) { create(:group, path: 'foo') }
 
   before do

ee/spec/features/issues/csv_spec.rb

@@ -6,7 +6,7 @@ describe 'Issues csv' do
   let(:milestone) { create(:milestone, title: 'v1.0', project: project) }
   let(:idea_label) { create(:label, project: project, title: 'Idea') }
   let(:feature_label) { create(:label, project: project, title: 'Feature', priority: 10) }
-  let!(:issue)  { create(:issue, project: project, author: user) }
+  let!(:issue) { create(:issue, project: project, author: user) }
 
   before do
     sign_in(user)

ee/spec/features/projects/pipelines/pipeline_spec.rb

@@ -47,7 +47,7 @@ describe 'Pipeline', :js do
   end
 
   describe 'GET /:project/pipelines/:id/licenses' do
-    let(:pipeline) {  create(:ci_pipeline, project: project, ref: 'master', sha: project.commit.id) }
+    let(:pipeline) { create(:ci_pipeline, project: project, ref: 'master', sha: project.commit.id) }
 
     before do
       stub_licensed_features(license_management: true)

ee/spec/features/promotion_spec.rb

@@ -7,7 +7,7 @@ describe 'Promotions', :js do
   let(:group) { create(:group) }
   let(:project) { create(:project, :repository, namespace: group) }
   let(:milestone) { create(:milestone, project: project, start_date: Date.today, due_date: 7.days.from_now) }
-  let!(:issue)  { create(:issue, project: project, author: user) }
+  let!(:issue) { create(:issue, project: project, author: user) }
   let(:otherproject) { create(:project, :repository, namespace: otherdeveloper.namespace) }
 
   describe 'if you have a license' do

ee/spec/finders/epics_finder_spec.rb

 require 'spec_helper'
 
 describe EpicsFinder do
-  let(:user) { create(:user)  }
-  let(:search_user) { create(:user)  }
+  let(:user) { create(:user) }
+  let(:search_user) { create(:user) }
   let(:group) { create(:group, :private) }
   let(:another_group) { create(:group) }
   let!(:epic1) { create(:epic, :opened, group: group, title: 'This is awesome epic', created_at: 1.week.ago) }