Skip to content

Add a link to the Kubernetes guidelines on code reviews.

username-removed-394049 requested to merge afolson/gitlab-ee:patch-1 into master

See the general Documentation guidelines http://docs.gitlab.com/ce/development/doc_styleguide.html

What does this MR do?

This MR adds a link to a great guide from Kubernetes. The guide contains excellent tips for getting MRs reviewed faster.

Discussion: https://gitlab.slack.com/archives/marketing/p1481072014000160

cc @sytses

Moving docs to a new location?

See the guidelines: http://docs.gitlab.com/ce/development/doc_styleguide.html#changing-document-location

  • Make sure the old link is not removed and has its contents replaced with a link to the new location.
  • Make sure internal links pointing to the document in question are not broken.
  • Search and replace any links referring to old docs in GitLab Rails app, specifically under the app/views/ directory.
  • If working on CE, submit an MR to EE with the changes as well.

Link to CE MR: https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/7966

Merge request reports