Age | Commit message (Collapse) | Author | |
---|---|---|---|
2018-05-30 | Fixed broken link to build team in documentation | Eddie Stubbington | |
2018-05-11 | Add Code Review documentation that maintainers also perform code review | Robert Speicher | |
2018-05-11 | Move footnote from CONTRIBUTING.md to doc/development/code_review.md | Robert Speicher | |
A section was moved in f644b8c8 but this got lost. | |||
2017-08-01 | Merge branch 'master' into 'docs-specific-review-examples' | Clement Ho | |
# Conflicts: # doc/development/code_review.md | |||
2017-08-01 | Add code review guidelines related to Build [CI skip]. | Marin Jankovski | |
2017-07-31 | Add note about UX paradigm | Clement Ho | |
2017-07-31 | Add specific code review guidelines | Clement Ho | |
2017-07-24 | Add GitLab-specific concerns to code review guide | Sean McGivern | |
2017-05-09 | Fix links in code review docs | Sean McGivern | |
2017-05-04 | Improving copy of CONTRIBUTING.md, PROCESS.md, and code_review.md | Rémy Coutable | |
Signed-off-by: Rémy Coutable <remy@rymai.me> | |||
2017-05-03 | Improve the Code review guidelines documentation | Rémy Coutable | |
Signed-off-by: Rémy Coutable <remy@rymai.me> | |||
2017-02-15 | Rename builds to jobs in docs | Achilleas Pipinellis | |
[ci skip] | |||
2017-01-13 | Rename endboss -> maintainer, miniboss -> reviewer | Sean McGivern | |
We want to describe these roles in a way that is more understandable to people not familiar with GitLab. | |||
2016-12-05 | Merge branch 'fix/rename-mwbs-to-merge-when-pipeline-succeeds' into 'master' | Kamil Trzciński | |
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-02 | Remove change to MWBS in code review guideline | Grzegorz Bizon | |
[ci skip] | |||
2016-12-01 | Copy-edit text about right balance in code reviews | Grzegorz Bizon | |
[ci skip] | |||
2016-11-30 | Extend code review docs with chapter about the right balance | Grzegorz Bizon | |
2016-11-21 | Update some docs to reflect MWPS name change | Grzegorz Bizon | |
2016-10-07 | Fix typo and add he MWBS accronym for "Merge When Build Succeeds" | Rémy Coutable | |
Signed-off-by: Rémy Coutable <remy@rymai.me> | |||
2016-10-07 | Improve the contribution and MR review guide | Rémy Coutable | |
Signed-off-by: Rémy Coutable <remy@rymai.me> | |||
2016-04-11 | Re-add a note about sarcasm to the Code Review guide | Robert Speicher | |
[ci skip] | |||
2016-04-10 | First pass at a Code Review guide | Robert Speicher | |
Largely borrowed from thoughtbot's code review guide, so attribution is included. [ci skip] |