This reverts the addition of the "goldiloader" Gem and all use of it.
While this Gem is very promising it's causing a variety of problems on
GitLab.com due to it eager-loading too much data in places where we
don't expect/can handle this. At least for the time being this means we
have to go back to manually fixing N+1 query problems, but at least
those should not cause a negative impact on availability.

Fix security products examples

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

CE upstream - 2018-04-17 20:56 UTC

Closes gitaly#401 et gitlab-org/release/docs#9

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

EE Port of CE!18422

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

Ee blackst0ne rails5 add safe params helper

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

Signed-off-by: Rémy Coutable <remy@rymai.me>

EE: Resolve "Make a Rubocop that forbids returning from a block"

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

Avoid wrong closing dates being caught by the query on Burndown charts

Closes gitlab-com/support-forum#3273

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

Update docs subgroup issues in group board

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

Remove differences with CE version of update_pages_service.rb

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

Usage ping for projects_reporting_ci_cd_back_to_github

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

Merge branch 'jej/5598-documentation-changelog-for-branch-unprotection-restrictions-api' into 'master'

First part of "Documentation & Changelog for Branch Unprotection Restrictions & API"

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

Merge branch '42889-avoid-return-inside-block' of gitlab.com:jacopo-beschi/gitlab-ce into 42889-avoid-return-inside-block-ee

* '42889-avoid-return-inside-block' of gitlab.com:jacopo-beschi/gitlab-ce:
  Uses the rubocop helpers expect_offense and expect_no_offenses
  Updates after 4th review
  Updates after 3rd review
  Updates after 2nd review
  Fixes cop errors
  Adds AvoidBreakFromStrongMemoize Cop
  Minor review changes
  Adds Rubocop rule to avoid returning from a block

Rails 5.0 requires to explicitly permit attributes when building a URL
using current `params` object.

The `safe_params` helper allows developers to just call `safe_params.merge(...)`
instead of manually adding `permit` to every call.

https://github.com/rails/rails/pull/20868

Avoid creating superfluous namespaces on Geo secondaries during sync

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

# Conflicts:
#	doc/README.md
#	doc/user/project/merge_requests/index.md
#	doc/user/project/web_ide/index.md
#	locale/gitlab.pot

[ci skip]

Docs: add product badges to docs

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

Docs: update Web IDE > CE (10.7)

See merge request gitlab-org/gitlab-ce!18433

Docs: add product badges to docs

Closes #42631

See merge request gitlab-org/gitlab-ce!18408

Geo: add extra reconfigure step in docs to enable FDW

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

Move Gitaly RefExists check to OPT_OUT

Closes gitaly#401

See merge request gitlab-org/gitlab-ce!18333

CE upstream - 2018-04-17 00:24 UTC

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

add initial dev task template for working on a security issue

Closes gitlab-org/release/docs#9

See merge request gitlab-org/gitlab-ce!18133