forked from qooxdoo/qooxdoo
-
Notifications
You must be signed in to change notification settings - Fork 0
/
.pullapprove.yml
64 lines (58 loc) · 1.92 KB
/
.pullapprove.yml
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
version: 3
# see https://docs.pullapprove.com/config/groups/
groups:
# source code documentation change only: 2 approvals necessary
documentation:
conditions:
- "'documentation' in labels"
reviews:
required: 2 # 2 approvals from this group are required
request: 5 # 5 review request will be sent at a time
reviewers:
teams:
- pr-mergers
# emergency hotfix: 2 approvals necessary
hotfix:
conditions:
- "'hotfix' in labels"
reviews:
required: 2
request: 5
reviewers:
teams:
- pr-mergers
# default procedure: 5 approvals required
default:
conditions:
# this group is asked to review if no previous groups match this PR
- "not groups.active"
reviews:
required: 5
request: 5
reviewers:
teams:
- pr-mergers
# see https://docs.pullapprove.com/config/pullapprove-conditions/
pullapprove_conditions:
- condition: "base.ref == 'master'"
unmet_status: success
explanation: "Review not required unless merging to master"
- condition: "'*travis*' in statuses.successful"
unmet_status: "failure"
explanation: "Tests must pass before review starts"
- condition: "'WIP' not in title"
unmet_status: pending
explanation: "Work in progress - review not yet required"
- condition: "'do not merge' not in labels"
unmet_status: pending
explanation: "PR should not be merged yet - review not yet required"
notifications:
- when: pull_request.opened
if: "author_association == 'FIRST_TIME_CONTRIBUTOR'"
comment: |
Hey @{{ author }}, thanks for the PR! The review will start once
the tests and CI checks have passed. If they don't, please review
the logs and try to fix the issues (ask for help if you can't
figure it out). A reviewer will be assigned once the tests are
passing and they'll walk you through getting the PR finished
and merged.