summaryrefslogtreecommitdiff
path: root/doc/development/code_review.md
Commit message (Collapse)AuthorAgeFilesLines
* Merge branch 'master' into 'docs-specific-review-examples'docs-specific-review-examplesClement Ho2017-08-011-0/+3
|\ | | | | | | # Conflicts: # doc/development/code_review.md
| * Add code review guidelines related to Build [CI skip].Marin Jankovski2017-08-011-0/+3
| |
* | Add note about UX paradigmClement Ho2017-07-311-0/+2
| |
* | Add specific code review guidelinesadd-specific-review-examplesClement Ho2017-07-311-0/+8
|/
* Add GitLab-specific concerns to code review guideSean McGivern2017-07-241-0/+49
|
* Fix links in code review docsSean McGivern2017-05-091-5/+10
|
* Improving copy of CONTRIBUTING.md, PROCESS.md, and code_review.mddocs/30141-improve-contributing-guideRémy Coutable2017-05-041-3/+0
| | | | Signed-off-by: Rémy Coutable <remy@rymai.me>
* Improve the Code review guidelines documentationRémy Coutable2017-05-031-5/+39
| | | | Signed-off-by: Rémy Coutable <remy@rymai.me>
* Rename builds to jobs in docsAchilleas Pipinellis2017-02-151-1/+1
| | | | [ci skip]
* Rename endboss -> maintainer, miniboss -> reviewerno-more-bossesSean McGivern2017-01-131-6/+6
| | | | | We want to describe these roles in a way that is more understandable to people not familiar with GitLab.
* Merge branch 'fix/rename-mwbs-to-merge-when-pipeline-succeeds' into 'master' Kamil Trzciński2016-12-051-2/+2
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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
| * Update some docs to reflect MWPS name changeGrzegorz Bizon2016-11-211-2/+2
| |
* | Remove change to MWBS in code review guidelineGrzegorz Bizon2016-12-021-1/+1
| | | | | | | | [ci skip]
* | Copy-edit text about right balance in code reviewsdoc/extend-code-review-guidelinesGrzegorz Bizon2016-12-011-10/+10
| | | | | | | | [ci skip]
* | Extend code review docs with chapter about the right balanceGrzegorz Bizon2016-11-301-1/+27
|/
* Fix typo and add he MWBS accronym for "Merge When Build Succeeds"improve-contributingRémy Coutable2016-10-071-4/+4
| | | | Signed-off-by: Rémy Coutable <remy@rymai.me>
* Improve the contribution and MR review guideRémy Coutable2016-10-071-0/+7
| | | | Signed-off-by: Rémy Coutable <remy@rymai.me>
* Re-add a note about sarcasm to the Code Review guiders-code-review-guideRobert Speicher2016-04-101-0/+3
| | | | [ci skip]
* First pass at a Code Review guideRobert Speicher2016-04-091-0/+75
Largely borrowed from thoughtbot's code review guide, so attribution is included. [ci skip]