Justin Wood | bc0bd6f | 2020-06-18 13:30:47 -0700 | [diff] [blame] | 1 | version: 3 |
| 2 | |
Justin Wood | f7b88a1 | 2020-06-19 11:56:57 -0700 | [diff] [blame] | 3 | # https://developer.github.com/v3/previews/#draft-pull-requests |
Justin Wood | 4fa4211 | 2020-06-19 11:25:14 -0700 | [diff] [blame] | 4 | github_api_version: "shadow-cat-preview" |
| 5 | |
| 6 | ############################################################ |
Justin Wood | 8839e4c | 2021-10-22 17:27:05 -0700 | [diff] [blame] | 7 | # Overrides |
Justin Wood | 4fa4211 | 2020-06-19 11:25:14 -0700 | [diff] [blame] | 8 | ############################################################ |
| 9 | |
Justin Wood | 8839e4c | 2021-10-22 17:27:05 -0700 | [diff] [blame] | 10 | overrides: |
| 11 | - if: "'hotfix' in labels" |
| 12 | status: success |
| 13 | explanation: "Hotfix label added, bypassing reviews" |
Justin Wood | 06747fd | 2020-07-01 11:36:15 -0700 | [diff] [blame] | 14 | |
| 15 | ############################################################ |
Justin Wood | 4fa4211 | 2020-06-19 11:25:14 -0700 | [diff] [blame] | 16 | # Draft PRs |
| 17 | ############################################################ |
Justin Wood | 8839e4c | 2021-10-22 17:27:05 -0700 | [diff] [blame] | 18 | - if: "draft or 'WIP' in title" |
| 19 | status: pending |
| 20 | explanation: "PR is draft, pending review" |
| 21 | - if: "draft" |
| 22 | status: pending |
| 23 | explanation: "PR is draft, pending review" |
Justin Wood | 4fa4211 | 2020-06-19 11:25:14 -0700 | [diff] [blame] | 24 | |
Justin Wood | 8839e4c | 2021-10-22 17:27:05 -0700 | [diff] [blame] | 25 | ############################################################ |
| 26 | # License Checks |
| 27 | ############################################################ |
| 28 | - if: "'*license/cla*' not in statuses.successful" |
| 29 | status: pending |
| 30 | explanation: "CLA must be agreed to by all contributors" |
Justin Wood | 4fa4211 | 2020-06-19 11:25:14 -0700 | [diff] [blame] | 31 | |
| 32 | ############################################################ |
| 33 | # Conditions to Skip Review |
| 34 | ############################################################ |
Justin Wood | 8839e4c | 2021-10-22 17:27:05 -0700 | [diff] [blame] | 35 | - if: "base.ref != 'master'" |
| 36 | status: success |
Justin Wood | 4fa4211 | 2020-06-19 11:25:14 -0700 | [diff] [blame] | 37 | explanation: "Review not required unless merging to master" |
| 38 | |
| 39 | ############################################################ |
Justin Wood | e7c4cd3 | 2021-02-02 19:25:23 -0800 | [diff] [blame] | 40 | # Required status checks |
| 41 | ############################################################ |
Justin Wood | 8839e4c | 2021-10-22 17:27:05 -0700 | [diff] [blame] | 42 | - if: "'*restyle*' not in statuses.successful" |
| 43 | status: failure |
Justin Wood | e7c4cd3 | 2021-02-02 19:25:23 -0800 | [diff] [blame] | 44 | explanation: "Style must be inline before reviewing can be complete" |
| 45 | |
Justin Wood | 8839e4c | 2021-10-22 17:27:05 -0700 | [diff] [blame] | 46 | ############################################################ |
| 47 | # Fast tracking |
| 48 | ############################################################ |
| 49 | - if: "'fast track' in labels" |
| 50 | status: success |
| 51 | explanation: "PR has been fast tracked, bypassing reviews" |
| 52 | |
Justin Wood | 4fa4211 | 2020-06-19 11:25:14 -0700 | [diff] [blame] | 53 | ############################################################ |
| 54 | # Notifications |
| 55 | ############################################################ |
| 56 | |
| 57 | notifications: |
| 58 | ############################################################ |
| 59 | # New contributors |
| 60 | ############################################################ |
| 61 | - when: pull_request.opened |
| 62 | if: "author_association == 'FIRST_TIME_CONTRIBUTOR'" |
| 63 | comment: | |
| 64 | Hey @{{ author }}, thanks for the PR! The review will start once |
| 65 | the tests and CI checks have passed. If they don't, please review |
| 66 | the logs and try to fix the issues (ask for help if you can't |
| 67 | figure it out). A reviewer will be assigned once the tests are |
| 68 | passing and they'll walk you through getting the PR finished |
| 69 | and merged. |
| 70 | |
Justin Wood | bc0bd6f | 2020-06-18 13:30:47 -0700 | [diff] [blame] | 71 | groups: |
| 72 | ############################################################ |
Justin Wood | 61060a8 | 2020-07-21 10:31:53 -0700 | [diff] [blame] | 73 | # Shared Reviewer Groups |
| 74 | ############################################################ |
| 75 | shared-reviewers-amazon: |
| 76 | type: optional |
| 77 | conditions: |
| 78 | - files.include('*') |
| 79 | reviewers: |
| 80 | teams: [reviewers-amazon] |
Rob Walker | 856636e | 2020-09-25 14:27:04 -0700 | [diff] [blame] | 81 | reviews: |
Justin Wood | 3379983 | 2021-09-15 17:57:46 -0700 | [diff] [blame] | 82 | request: 10 |
Justin Wood | 61060a8 | 2020-07-21 10:31:53 -0700 | [diff] [blame] | 83 | shared-reviewers-apple: |
| 84 | type: optional |
| 85 | conditions: |
| 86 | - files.include('*') |
| 87 | reviewers: |
| 88 | teams: [reviewers-apple] |
Rob Walker | 856636e | 2020-09-25 14:27:04 -0700 | [diff] [blame] | 89 | reviews: |
Justin Wood | 3379983 | 2021-09-15 17:57:46 -0700 | [diff] [blame] | 90 | request: 10 |
Justin Wood | 61060a8 | 2020-07-21 10:31:53 -0700 | [diff] [blame] | 91 | shared-reviewers-comcast: |
| 92 | type: optional |
| 93 | conditions: |
| 94 | - files.include('*') |
| 95 | reviewers: |
| 96 | teams: [reviewers-comcast] |
Rob Walker | 856636e | 2020-09-25 14:27:04 -0700 | [diff] [blame] | 97 | reviews: |
Justin Wood | 3379983 | 2021-09-15 17:57:46 -0700 | [diff] [blame] | 98 | request: 10 |
Justin Wood | 8839e4c | 2021-10-22 17:27:05 -0700 | [diff] [blame] | 99 | # shared-reviewers-espressif: |
| 100 | # type: optional |
| 101 | # conditions: |
| 102 | # - files.include('*') |
| 103 | # reviewers: |
| 104 | # teams: [reviewers-espressif] |
| 105 | # reviews: |
| 106 | # request: 10 |
Justin Wood | 61060a8 | 2020-07-21 10:31:53 -0700 | [diff] [blame] | 107 | shared-reviewers-google: |
| 108 | type: optional |
| 109 | conditions: |
| 110 | - files.include('*') |
| 111 | reviewers: |
| 112 | teams: [reviewers-google] |
Rob Walker | 856636e | 2020-09-25 14:27:04 -0700 | [diff] [blame] | 113 | reviews: |
Justin Wood | 3379983 | 2021-09-15 17:57:46 -0700 | [diff] [blame] | 114 | request: 10 |
Justin Wood | 8839e4c | 2021-10-22 17:27:05 -0700 | [diff] [blame] | 115 | # shared-reviewers-lg: |
| 116 | # type: optional |
| 117 | # conditions: |
| 118 | # - files.include('*') |
| 119 | # reviewers: |
| 120 | # teams: [reviewers-lg] |
| 121 | # reviews: |
| 122 | # request: 10 |
Justin Wood | 4e87eb8 | 2021-03-09 14:52:30 -0800 | [diff] [blame] | 123 | shared-reviewers-nordic: |
| 124 | type: optional |
| 125 | conditions: |
| 126 | - files.include('*') |
| 127 | reviewers: |
| 128 | teams: [reviewers-nordic] |
| 129 | reviews: |
Justin Wood | 3379983 | 2021-09-15 17:57:46 -0700 | [diff] [blame] | 130 | request: 10 |
Justin Wood | 61060a8 | 2020-07-21 10:31:53 -0700 | [diff] [blame] | 131 | shared-reviewers-samsung: |
| 132 | type: optional |
| 133 | conditions: |
| 134 | - files.include('*') |
| 135 | reviewers: |
| 136 | teams: [reviewers-samsung] |
Rob Walker | 856636e | 2020-09-25 14:27:04 -0700 | [diff] [blame] | 137 | reviews: |
Justin Wood | 3379983 | 2021-09-15 17:57:46 -0700 | [diff] [blame] | 138 | request: 10 |
Justin Wood | 4e87eb8 | 2021-03-09 14:52:30 -0800 | [diff] [blame] | 139 | shared-reviewers-silabs: |
| 140 | type: optional |
| 141 | conditions: |
| 142 | - files.include('*') |
| 143 | reviewers: |
| 144 | teams: [reviewers-silabs] |
| 145 | reviews: |
Justin Wood | 3379983 | 2021-09-15 17:57:46 -0700 | [diff] [blame] | 146 | request: 10 |
Justin Wood | 2addf00 | 2021-09-10 08:33:23 -0700 | [diff] [blame] | 147 | # shared-reviewers-tcl: |
| 148 | # type: optional |
| 149 | # conditions: |
| 150 | # - files.include('*') |
| 151 | # reviewers: |
| 152 | # teams: [reviewers-tcl] |
| 153 | # reviews: |
| 154 | # request: 1 |
Justin Wood | 61060a8 | 2020-07-21 10:31:53 -0700 | [diff] [blame] | 155 | |
| 156 | ############################################################ |
Justin Wood | bc0bd6f | 2020-06-18 13:30:47 -0700 | [diff] [blame] | 157 | # Base Required Reviewers |
| 158 | ############################################################ |
| 159 | required-reviewers: |
Justin Wood | b02d517 | 2020-06-19 12:00:56 -0700 | [diff] [blame] | 160 | description: > |
| 161 | [Required |
| 162 | Reviewers](https://github.com/project-chip/connectedhomeip/blob/master/CONTRIBUTING.md#review-requirements) |
Justin Wood | a2c0c6f | 2020-06-19 12:03:41 -0700 | [diff] [blame] | 163 | This is the main group of required reviews for general pull |
| 164 | requests. |
Justin Wood | bc0bd6f | 2020-06-18 13:30:47 -0700 | [diff] [blame] | 165 | type: required |
Justin Wood | 61060a8 | 2020-07-21 10:31:53 -0700 | [diff] [blame] | 166 | requirements: |
| 167 | - len(groups.approved.include('shared-reviewers-*')) >= 3 |
Justin Wood | bc0bd6f | 2020-06-18 13:30:47 -0700 | [diff] [blame] | 168 | reviews: |
Justin Wood | 61060a8 | 2020-07-21 10:31:53 -0700 | [diff] [blame] | 169 | required: 0 |
Justin Wood | 2d579dd | 2021-05-17 18:18:33 -0700 | [diff] [blame] | 170 | labels: |
| 171 | approved: "review - approved" |
| 172 | pending: "review - pending" |
| 173 | rejected: "review - changed requested" |