Welcome to mirror list, hosted at ThFree Co, Russian Federation.

gitlab.com/gitlab-org/gitlab-foss.git - Unnamed repository; edit this file 'description' to name the repository.
summaryrefslogtreecommitdiff
AgeCommit message (Collapse)Author
2018-05-30Fixed broken link to build team in documentationEddie Stubbington
2018-05-11Add Code Review documentation that maintainers also perform code reviewRobert Speicher
2018-05-11Move footnote from CONTRIBUTING.md to doc/development/code_review.mdRobert Speicher
A section was moved in f644b8c8 but this got lost.
2017-08-01Merge branch 'master' into 'docs-specific-review-examples'Clement Ho
# Conflicts: # doc/development/code_review.md
2017-08-01Add code review guidelines related to Build [CI skip].Marin Jankovski
2017-07-31Add note about UX paradigmClement Ho
2017-07-31Add specific code review guidelinesClement Ho
2017-07-24Add GitLab-specific concerns to code review guideSean McGivern
2017-05-09Fix links in code review docsSean McGivern
2017-05-04Improving copy of CONTRIBUTING.md, PROCESS.md, and code_review.mdRémy Coutable
Signed-off-by: Rémy Coutable <remy@rymai.me>
2017-05-03Improve the Code review guidelines documentationRémy Coutable
Signed-off-by: Rémy Coutable <remy@rymai.me>
2017-02-15Rename builds to jobs in docsAchilleas Pipinellis
[ci skip]
2017-01-13Rename endboss -> maintainer, miniboss -> reviewerSean McGivern
We want to describe these roles in a way that is more understandable to people not familiar with GitLab.
2016-12-05Merge 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-02Remove change to MWBS in code review guidelineGrzegorz Bizon
[ci skip]
2016-12-01Copy-edit text about right balance in code reviewsGrzegorz Bizon
[ci skip]
2016-11-30Extend code review docs with chapter about the right balanceGrzegorz Bizon
2016-11-21Update some docs to reflect MWPS name changeGrzegorz Bizon
2016-10-07Fix 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-07Improve the contribution and MR review guideRémy Coutable
Signed-off-by: Rémy Coutable <remy@rymai.me>
2016-04-11Re-add a note about sarcasm to the Code Review guideRobert Speicher
[ci skip]
2016-04-10First pass at a Code Review guideRobert Speicher
Largely borrowed from thoughtbot's code review guide, so attribution is included. [ci skip]