Skip to content
Snippets Groups Projects
Commit b1446832 authored by Javyer's avatar Javyer
Browse files

updated review-bot.yml minApprovals convention

Renamed `min_approvals` to `minApprovals`. A breaking change in paritytech/review-bot#86 which was done to standarize all the cases (so now everything is camelCase)
parent 677610ba
No related merge requests found
Pipeline #397068 failed with stages
in 51 minutes and 21 seconds
......@@ -10,7 +10,7 @@ rules:
- ^\.cargo/.*
exclude:
- ^./gitlab/pipeline/zombienet.*
min_approvals: 2
minApprovals: 2
type: basic
teams:
- ci
......@@ -27,7 +27,7 @@ rules:
exclude:
- ^polkadot/runtime\/(kusama|polkadot)\/src\/weights\/.+\.rs$
- ^substrate\/frame\/.+\.md$
min_approvals: 1
minApprovals: 1
allowedToSkipRule:
teams:
- core-devs
......@@ -54,7 +54,7 @@ rules:
- ^\.gitlab/.*
- ^\.config/nextest.toml
- ^\.cargo/.*
min_approvals: 2
minApprovals: 2
type: basic
teams:
- core-devs
......@@ -70,10 +70,10 @@ rules:
- ^cumulus/parachains/common/src/[^/]+\.rs$
type: and-distinct
reviewers:
- min_approvals: 1
- minApprovals: 1
teams:
- locks-review
- min_approvals: 1
- minApprovals: 1
teams:
- polkadot-review
......@@ -83,7 +83,7 @@ rules:
condition:
include:
- ^bridges/.*
min_approvals: 1
minApprovals: 1
teams:
- bridges-core
......@@ -95,10 +95,10 @@ rules:
- ^substrate/frame/(?!.*(nfts/.*|uniques/.*|babe/.*|grandpa/.*|beefy|merkle-mountain-range/.*|contracts/.*|election|nomination-pools/.*|staking/.*|aura/.*))
type: "and"
reviewers:
- min_approvals: 2
- minApprovals: 2
teams:
- core-devs
- min_approvals: 1
- minApprovals: 1
teams:
- frame-coders
......@@ -107,15 +107,14 @@ rules:
condition:
include:
- review-bot\.yml
min_approvals: 2
type: "and"
reviewers:
- min_approvals: 1
- minApprovals: 1
teams:
- opstooling
- min_approvals: 1
- minApprovals: 1
teams:
- locks-review
- min_approvals: 1
- minApprovals: 1
teams:
- ci
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment