Codestyle changes to make sure we have space around operators (EE)

This will introduce fixes for gitlab-org/gitlab-ce#17398 so after
gitlab-org/gitlab-ce!4112 gets merged and CE synced to EE, we don't
have failing builds.

See merge request !388

Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>

Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>

Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>

These will fail until #558 is taken care of.

Updated EE CHANGELOG for 8.7.5

See merge request !395

[ci skip]

Fix skipping pages when restoring backups

Fixes https://gitlab.com/gitlab-org/gitlab-ee/issues/407

See merge request !375

This has simply been copied over from CE to prevent any future
conflicts.

Prepare update guide for 8.8

See merge request !391

Reduce emails-on-push HTML size by using a simple monospace font

With this simple change we reduce the size dramatically by reducing every inlined CSS instance of this:

font-family: 'Menlo', 'Liberation Mono', 'Consolas', 'DejaVu Sans Mono', 'Ubuntu Mono', 'Courier New', 'andale mono', 'lucida console', monospace"

to this:

font-family: monospace

Before:

![image](/uploads/6ce445102dfe5bfcdf9cf3479218b0ad/image.png)

After:

![image](/uploads/8e3a4216815986543ece75483d7f9063/image.png)

Helps gitlab-org/gitlab-ce#15178

See merge request !384

Updated EE CHANGELOG per 8-7-stable-ee

See merge request !390

[ci skip]

Updated 8.7.4 CHANGELOG entries

See merge request !4116

[ci skip]

Enable Rubocop Casecmp Performance Cop.

Also fixes the errors caused by enabling the cop.

`casecmp` is more performant than `.downcase` and `==`.

See also: https://github.com/bbatsov/rubocop/blob/master/lib/rubocop/cop/performance/casecmp.rb

See merge request !3957

Enable the StringReplacement cop.

Also fix one use of `gsub` that would be faster as `delete`.

Use `tr` instead of `gsub` when you are replacing the same number of characters. Use `delete` instead of `gsub` when you are deleting characters.

See merge request !3960

Fix LDAP access level spillover bug

See merge request !499

ES: Refactoring

Fixes https://gitlab.com/gitlab-org/gitlab-ee/issues/516

See merge request !365

Removes jquery-scrollto-rails.

Instead the script is now included directly from the vendor directory.

Resolves #17166. Working towards #14286 and merging !3775.

The gem was on 1.4.3.1, here's the diff between that tag and 2.1.2 (the version I'm using in this MR): https://github.com/flesler/jquery.scrollTo/compare/1.4.3...2.1.2

There aren't any notable breaking changes that I noticed, and I didn't see any JS errors after testing a bunch of pages.

cc: @jschatz1 

See merge request !4088

Also fix one use of `gsub` that would be faster as `delete`.

Use `tr` instead of `gsub` when you are replacing the same number of
characters. Use `delete` instead of `gsub` when you are deleting
characters.

Instead the script is now included directly from the vendor directory.

Resolves #17166.

Also fixes the errors caused by enabling the cop. casecmp is more performant than `.downcase` and `==`.

Restore "r" shortcut

Fixes "Reply with Selected Text" shortcut

See merge request !4041

Add eager load paths to help prevent dependency load issues with Sidekiq workers

_Originally opened at !3545 by @stanhu._

- - -

Relevant resources:

- https://github.com/mperham/sidekiq/wiki/FAQ#why-doesnt-sidekiq-autoload-my-rails-application-code
- https://github.com/mperham/sidekiq/issues/1281#issuecomment-27129904
- http://blog.arkency.com/2014/11/dont-forget-about-eager-load-when-extending-autoload
- https://github.com/rails/rails/blob/52ce6ece8c8f74064bb64e0a0b1ddd83092718e1/railties/lib/rails/engine.rb#L472-L479
- https://github.com/rails/rails/blob/v4.2.6/railties/lib/rails/paths.rb

Attempts to address #3661, #11896, #12769, #13521, #14131, #14589, #14759, #14825.

See merge request !3724