delta/gitlab/gitlab-ce.git/CONTRIBUTING.md, branch fix-commit-status

Merge branch 'mr-performance-guides' into 'master'

2016-08-31T12:51:36+00:00


Added performance guidelines for new MRs

## What does this MR do?

This MR adds a set of guides that should be followed by merge request authors.

## Are there points in the code the reviewer needs to double check?

Spelling, grammar, etc

## Why was this MR needed?

There is no set of guidelines one should follow when submitting merge requests. This leads to developers at times disregarding performance. This in turn results in performance specialists having to clean up the mess, or production engineers being woken up in the middle of the night because the database is on fire.

## Does this MR meet the acceptance criteria?

- [x] [Documentation created/updated](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/doc/development/doc_styleguide.md)
- Tests
  - [x] All builds are passing
- [x] Conform by the [style guides](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/CONTRIBUTING.md#style-guides)
- [x] Branch has no merge conflicts with `master` (if you do - rebase it please)
- [x] [Squashed related commits together](https://git-scm.com/book/en/Git-Tools-Rewriting-History#Squashing-Commits)

cc @DouweM @rspeicher @pcarranza @dzaporozhets 

See merge request !5905

Link to the issue templates in Contributing guide

2016-08-25T14:48:08+00:00

Change to use a better title style

Add backticks to code segments

Spelling and Grammar

Reflect template addition in the contributing guide

2016-08-24T14:00:33+00:00

Adjust bug template to adhere to standard bug report format

* https://developer.mozilla.org/en-US/docs/Mozilla/QA/Bug_writing_guidelines
* Guidelines use the following format

- Summary
- Steps to Reproduce
- Expected Behaviour
- Actual Behaviour

Added performance guidelines for new MRs

2016-08-22T12:32:25+00:00

These guidelines cover the performance requirement for newly submitted
merge requests. These guidelines are put in to place to prevent merge
requests from negatively impacting GitLab performance as much as
possible.

Update contribution acceptance criteria with tests requirements

2016-08-19T09:24:51+00:00

Add test requirement for new files

2016-08-15T15:43:04+00:00

Merge branch 'dz-newlines-styleguide' into 'master'

2016-08-08T23:29:56+00:00


Add newlines styleguide for Ruby code 

See merge request !5636

Add migration-related tips to the "Merge Request Guidelines" doc

2016-08-08T03:13:19+00:00

[ci-skip]

Add newlines styleguide for Ruby code

2016-08-05T09:19:19+00:00

Merge branch 'coffescript_docs_migration' into 'master'

2016-08-04T11:41:52+00:00


Update docs for CoffeeScript -> JavaScript.

## What does this MR do?
Removes some CoffeeScript docs and adds more JS ones

## Are there points in the code the reviewer needs to double check?
No

## Why was this MR needed?
To migrate some docs

## What are the relevant issue numbers?
#20098

## Screenshots (if relevant)
None

## Does this MR meet the acceptance criteria?

- [x] [CHANGELOG](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/CHANGELOG) entry added
- [x] [Documentation created/updated](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/doc/development/doc_styleguide.md)
- [ ] API support added
- Tests
  - [ ] Added for this feature/bug
  - [ ] All builds are passing
- [x] Conform by the [style guides](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/CONTRIBUTING.md#style-guides)
- [x] Branch has no merge conflicts with `master` (if you do - rebase it please)
- [x] [Squashed related commits together](https://git-scm.com/book/en/Git-Tools-Rewriting-History#Squashing-Commits)

See merge request !5433