Skip to content

Commit

Permalink
Enhancement: Add support for checks key in required_status_checks
Browse files Browse the repository at this point in the history
  • Loading branch information
localheinz committed Feb 8, 2022
1 parent 94476e4 commit 80a6968
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 2 deletions.
4 changes: 3 additions & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -144,8 +144,10 @@ branches:
required_status_checks:
# Required. Require branches to be up to date before merging.
strict: true
# Required. The list of status checks to require in order to merge into this branch
# 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.
contexts: []
# The list of status checks to require in order to merge into this branch
checks: []
# Required. Enforce all configured restrictions for administrators. Set to true to enforce required status checks for repository administrators. Set to null to disable.
enforce_admins: true
# Prevent merge commits from being pushed to matching branches
Expand Down
7 changes: 6 additions & 1 deletion test/unit/lib/mergeArrayByName.test.js
Original file line number Diff line number Diff line change
Expand Up @@ -44,6 +44,7 @@ describe('mergeArrayByName', () => {
required_status_checks:
strict: true
contexts: []
checks: []
enforce_admins: false
restrictions:
`)
Expand All @@ -67,7 +68,11 @@ describe('mergeArrayByName', () => {
require_code_owner_reviews: true,
dismissal_restrictions: {}
},
required_status_checks: { strict: true, contexts: [] },
required_status_checks: {
strict: true,
contexts: [],
checks: []
},
enforce_admins: false,
restrictions: null
}
Expand Down

0 comments on commit 80a6968

Please sign in to comment.