Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Merge branch 'master' into 'docs-specific-review-examples'docs-specific-review-examples | Clement Ho | 2017-08-01 | 1 | -0/+3 |
|\ | | | | | | | # Conflicts: # doc/development/code_review.md | ||||
| * | Add code review guidelines related to Build [CI skip]. | Marin Jankovski | 2017-08-01 | 1 | -0/+3 |
| | | |||||
* | | Add note about UX paradigm | Clement Ho | 2017-07-31 | 1 | -0/+2 |
| | | |||||
* | | Add specific code review guidelinesadd-specific-review-examples | Clement Ho | 2017-07-31 | 1 | -0/+8 |
|/ | |||||
* | Add GitLab-specific concerns to code review guide | Sean McGivern | 2017-07-24 | 1 | -0/+49 |
| | |||||
* | Fix links in code review docs | Sean McGivern | 2017-05-09 | 1 | -5/+10 |
| | |||||
* | Improving copy of CONTRIBUTING.md, PROCESS.md, and code_review.mddocs/30141-improve-contributing-guide | Rémy Coutable | 2017-05-04 | 1 | -3/+0 |
| | | | | Signed-off-by: Rémy Coutable <remy@rymai.me> | ||||
* | Improve the Code review guidelines documentation | Rémy Coutable | 2017-05-03 | 1 | -5/+39 |
| | | | | Signed-off-by: Rémy Coutable <remy@rymai.me> | ||||
* | Rename builds to jobs in docs | Achilleas Pipinellis | 2017-02-15 | 1 | -1/+1 |
| | | | | [ci skip] | ||||
* | Rename endboss -> maintainer, miniboss -> reviewerno-more-bosses | Sean McGivern | 2017-01-13 | 1 | -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ński | 2016-12-05 | 1 | -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 change | Grzegorz Bizon | 2016-11-21 | 1 | -2/+2 |
| | | |||||
* | | Remove change to MWBS in code review guideline | Grzegorz Bizon | 2016-12-02 | 1 | -1/+1 |
| | | | | | | | | [ci skip] | ||||
* | | Copy-edit text about right balance in code reviewsdoc/extend-code-review-guidelines | Grzegorz Bizon | 2016-12-01 | 1 | -10/+10 |
| | | | | | | | | [ci skip] | ||||
* | | Extend code review docs with chapter about the right balance | Grzegorz Bizon | 2016-11-30 | 1 | -1/+27 |
|/ | |||||
* | Fix typo and add he MWBS accronym for "Merge When Build Succeeds"improve-contributing | Rémy Coutable | 2016-10-07 | 1 | -4/+4 |
| | | | | Signed-off-by: Rémy Coutable <remy@rymai.me> | ||||
* | Improve the contribution and MR review guide | Rémy Coutable | 2016-10-07 | 1 | -0/+7 |
| | | | | Signed-off-by: Rémy Coutable <remy@rymai.me> | ||||
* | Re-add a note about sarcasm to the Code Review guiders-code-review-guide | Robert Speicher | 2016-04-10 | 1 | -0/+3 |
| | | | | [ci skip] | ||||
* | First pass at a Code Review guide | Robert Speicher | 2016-04-09 | 1 | -0/+75 |
Largely borrowed from thoughtbot's code review guide, so attribution is included. [ci skip] |