Skip to content

Commit 80a6968

Browse files
committed
Enhancement: Add support for checks key in required_status_checks
1 parent 94476e4 commit 80a6968

File tree

2 files changed

+9
-2
lines changed

2 files changed

+9
-2
lines changed

README.md

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -144,8 +144,10 @@ branches:
144144
required_status_checks:
145145
# Required. Require branches to be up to date before merging.
146146
strict: true
147-
# Required. The list of status checks to require in order to merge into this branch
147+
# Required. Deprecated. The list of status checks to require in order to merge into this branch. If any of these checks have recently been set by a particular GitHub App, they will be required to come from that app in future for the branch to merge. Use checks instead of contexts for more fine-grained control.
148148
contexts: []
149+
# The list of status checks to require in order to merge into this branch
150+
checks: []
149151
# Required. Enforce all configured restrictions for administrators. Set to true to enforce required status checks for repository administrators. Set to null to disable.
150152
enforce_admins: true
151153
# Prevent merge commits from being pushed to matching branches

test/unit/lib/mergeArrayByName.test.js

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -44,6 +44,7 @@ describe('mergeArrayByName', () => {
4444
required_status_checks:
4545
strict: true
4646
contexts: []
47+
checks: []
4748
enforce_admins: false
4849
restrictions:
4950
`)
@@ -67,7 +68,11 @@ describe('mergeArrayByName', () => {
6768
require_code_owner_reviews: true,
6869
dismissal_restrictions: {}
6970
},
70-
required_status_checks: { strict: true, contexts: [] },
71+
required_status_checks: {
72+
strict: true,
73+
contexts: [],
74+
checks: []
75+
},
7176
enforce_admins: false,
7277
restrictions: null
7378
}

0 commit comments

Comments
 (0)