Commit graph

9 commits

Author SHA1 Message Date
Kamil Trzciński
0f90fd639c Merge branch 'fix/rename-mwbs-to-merge-when-pipeline-succeeds' into 'master'
Rename MWBS service to Merge When Pipeline Succeeds

## What does this MR do?

This MR renames Merge When Build Succeeds feature to Merge When Pipeline Succeeds, because we recently changed the behavior, see https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/6675.

## Does this MR meet the acceptance criteria?

- [x] [CHANGELOG](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/CHANGELOG.md) entry added
- [x] [Documentation created/updated](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/doc/development/doc_styleguide.md)
- [x] All builds are passing

## What are the relevant issue numbers?

Closes #23354

See merge request !7135
2016-12-05 11:07:57 +00:00
Grzegorz Bizon
ebef1a8441 Remove change to MWBS in code review guideline
[ci skip]
2016-12-02 08:55:53 +01:00
Grzegorz Bizon
85c4aa4a67 Copy-edit text about right balance in code reviews
[ci skip]
2016-12-01 10:42:35 +01:00
Grzegorz Bizon
5b052605b7 Extend code review docs with chapter about the right balance 2016-11-30 11:23:09 +01:00
Grzegorz Bizon
c6a4f9fc5b Update some docs to reflect MWPS name change 2016-11-21 11:27:28 +01:00
Rémy Coutable
52ca9bf600
Fix typo and add he MWBS accronym for "Merge When Build Succeeds"
Signed-off-by: Rémy Coutable <remy@rymai.me>
2016-10-07 16:17:28 +02:00
Rémy Coutable
2f7e28d1f7
Improve the contribution and MR review guide
Signed-off-by: Rémy Coutable <remy@rymai.me>
2016-10-07 15:28:15 +02:00
Robert Speicher
0c6923e2d1 Re-add a note about sarcasm to the Code Review guide
[ci skip]
2016-04-10 22:54:44 -04:00
Robert Speicher
c7ec5929b1 First pass at a Code Review guide
Largely borrowed from thoughtbot's code review guide, so attribution is
included.

[ci skip]
2016-04-09 21:49:47 -04:00