0eb3d2f799
Add latest changes from gitlab-org/gitlab@master
2020-02-06 15:09:11 +00:00
ca05512007
Add latest changes from gitlab-org/gitlab@master
2020-02-04 18:08:50 +00:00
0434f38ef1
Add latest changes from gitlab-org/gitlab@master
2020-01-31 18:09:11 +00:00
30010b161d
Add latest changes from gitlab-org/gitlab@master
2020-01-08 03:08:05 +00:00
8dafc3b65a
Add latest changes from gitlab-org/gitlab@master
2020-01-07 18:07:34 +00:00
0ba52d8e77
Add latest changes from gitlab-org/gitlab@master
2019-12-28 09:08:05 +00:00
8f1f6b374b
Add latest changes from gitlab-org/gitlab@master
2019-11-25 12:06:13 +00:00
6026bddcd5
Add latest changes from gitlab-org/gitlab@master
2019-10-31 09:06:29 +00:00
3fe9588b1c
Add latest changes from gitlab-org/gitlab@master
2019-10-30 15:14:17 +00:00
77a7772c3b
Add latest changes from gitlab-org/gitlab@master
2019-10-08 09:06:09 +00:00
45482d5a27
Add latest changes from gitlab-org/gitlab@master
2019-09-27 12:06:07 +00:00
80f61b4035
Add latest changes from gitlab-org/gitlab@master
2019-09-18 14:02:45 +00:00
5939b09fd3
Add latest changes from gitlab-org/gitlab@master
2019-09-13 15:06:04 +00:00
a1852c6675
Fix broken link to the handbook
...
- Broken link to "How to work with the distribution team"
2019-09-09 17:33:20 -05:00
53275ad1a2
What to avoid in code review
2019-09-04 10:57:10 +10:00
fda560bdb4
Add Danger bot development documentation
...
This is overdue, and some of the suggestions are embarrassing, but
this is how we actually develop Danger today.
2019-09-03 00:23:52 +00:00
cd2a503db0
Add a section of examples
...
We have a fairly good guide to Code Reviews, but can be improved
by adding a few examples of what a good code review looks like
at GitLab, specifically ones where there is a bit of back and
forth, "nit-picking," etc. This would:
+ help set expectations of newly hired engineers around what our
process looks like when it is functioning what level of scrutiny
their code will be under
+ how we have technical conversations
+ show by example how after you're done crafting a solution, there
can still be extra work done either tidying up code and/or managing
the communication and conversations about your proposed MR
2019-07-25 14:02:10 +00:00
34a5f77e77
Document database review process
...
See https://gitlab.com/gitlab-com/gl-infra/infrastructure/issues/6069
2019-07-19 17:33:48 +00:00
7da80b2d36
Update numbered lists for docs standards
...
Ensure that all numbered lists use only 1. and no other numbers.
Also ensure that numbered lists use proper spacing.
2019-07-19 02:20:32 +00:00
746f547877
Fix unordered list spacing
...
Correct the spacing of unordered markdown lists
in docs, to maintain standards of documentation.
2019-07-18 01:15:58 +00:00
24a5fe6b14
Rename "discussions" to "threads" in docs
2019-07-15 06:10:32 +00:00
00780264a7
Update the installation link
...
The installation link has changed from /installation/ to /install/. This
commit will update all the remaining links that are missing.
2019-07-08 17:24:54 -05:00
942b38d5af
Require database reviews for migrations
2019-06-24 16:53:16 +00:00
8ec7ecf0bc
Use relative URLs in development docs
...
This is part of https://gitlab.com/gitlab-org/gitlab-ce/issues/61945
2019-05-27 11:13:40 -04:00
c009e23a16
Give reviewer roulette its own header in the docs
2019-04-30 10:03:53 +01:00
95dbc34e29
Refer to guidelines on when to assign MRs close to the freeze
2019-04-16 18:44:39 +02:00
0477cd00dd
Rewrite "Review turnaround time" section
2019-04-16 18:22:34 +02:00
58d3d9ce9c
Add ssot link to security reviews documentation
2019-03-25 16:56:51 +00:00
cbeab67a27
Apply suggestion to doc/development/code_review.md
2019-02-21 16:43:53 +00:00
6349a18588
Update code review guidelines
2019-02-21 12:07:30 +00:00
4e42168b09
Merge branch 'docs/zj-reviewer-guidelines' into 'master'
...
Add a soft SLA for reviewers and maintainers
See merge request gitlab-org/gitlab-ce!25192
2019-02-18 10:31:04 +00:00
633cb37e28
Improve language
2019-02-18 07:14:16 +00:00
77b2ecd2b1
Reviewer roulette via Danger
...
Make danger pick reviewers and maintainers at random, for feontend,
backend, database, etc, changes, whenever files belonging to those
teams get changed.
2019-02-13 16:41:28 +00:00
341556816c
Add a soft SLA for reviewers and maintainers
...
By setting expectations both the contributor and reviewer have more
certainty on what gets reviewed when, and at what speed changes could be
merged.
2019-02-13 13:07:09 +01:00
ceb7631ca5
Fix links to sections that were moved
2018-12-11 20:09:05 +00:00
bc3e68d537
Fix Red/Orange data link markdown
2018-11-30 15:07:06 +00:00
86bf2dd349
Merge branch 'docs/8608-update-approval-guidelines-to-include-red-orange-data-reviews' into 'master'
...
Add RED data security requirement to code review
See merge request gitlab-org/gitlab-ce!23426
2018-11-29 12:11:50 +00:00
f6e51e83a5
Merge branch 'mk/author-preparation-docs' into 'master'
...
Encourage MR author preparation in Code Review Guidelines documentation
See merge request gitlab-org/gitlab-ce!23360
2018-11-29 09:58:41 +00:00
62fd842c65
Add RED data security requirement to code review
...
closes #8608
2018-11-28 15:19:45 -05:00
59b4db7f2c
Encourage MR author preparation
2018-11-28 10:37:17 -08:00
cbfd6aced3
Left shift security in our workflow
2018-11-28 11:05:01 +00:00
367a0e8914
Clearer docs when it is OK for maintainer to squash
2018-11-27 11:55:01 +01:00
20146580a0
Resolve Markdown ordered lists not conforming to styleguide
2018-11-13 10:53:38 +10:00
a9b3f0811a
Includes the way approvals are handled to the documentation
2018-11-06 08:23:14 +00:00
2a631de547
Strongly recommend involving a domain expert, especially when in doubt.
2018-10-17 17:38:45 +00:00
a706b3735e
Merge branch 'master' into 'dm-document-role-maintainer'
...
# Conflicts:
# doc/development/code_review.md
2018-10-16 22:39:33 +00:00
90056ed25b
Clarify responsibilities of MR author and maintainer based on feedback.
2018-10-16 22:31:27 +00:00
5fa781ecf7
Fix "ready for review" link
...
Remove assignee_id=0 condition from search filters, we agreed
that just presence of "ready of review" label is sufficient
for filtering MRs, no need to unassign yourself.
2018-10-16 13:33:25 +02:00
189f366b9c
Add documentation how to assign a merge request for review
2018-10-15 13:16:46 +02:00
eb0ded1d90
Rewrite guidance on getting your merge request reviewed, approved, and merged
2018-10-10 10:48:14 +00:00
GitLab Bot
David H. Wilkins
Marin Jankovski
Nick Thomas
Kerri Miller
Toon Claes
Marcel Amirault
Michel Engelen
Cindy Pallares
Andreas Brandl
Achilleas Pipinellis
Sean McGivern
Douwe Maan
James Ritchey
Marcia Ramos
Marcia Ramos
Zeger-Jan van de Weg
Michael Kozono
Philippe Lafoucrière
Rémy Coutable
Philippe Lafoucrière
Dylan Griffith
Evan Read
Dimitrie Hoekstra
Douwe Maan
Jan Provaznik
Jarka Kadlecová