Skip to content

Commit

Permalink
Use []RequiredStatusCheck instead of []*RequiredStatusCheck
Browse files Browse the repository at this point in the history
  • Loading branch information
k24dizzle committed Jan 29, 2022
1 parent 1ee3619 commit 1002c13
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 9 deletions.
2 changes: 1 addition & 1 deletion github/repos.go
Expand Up @@ -882,7 +882,7 @@ type RequiredStatusChecks struct {
Contexts []string `json:"contexts,omitempty"`
// The list of status checks to require in order to merge into this
// branch.
Checks []*RequiredStatusCheck `json:"checks,omitempty"`
Checks []RequiredStatusCheck `json:"checks,omitempty"`
}

// RequiredStatusChecksRequest represents a request to edit a protected branch's status checks.
Expand Down
16 changes: 8 additions & 8 deletions github/repos_test.go
Expand Up @@ -1087,7 +1087,7 @@ func TestRepositoriesService_GetBranchProtection(t *testing.T) {
RequiredStatusChecks: &RequiredStatusChecks{
Strict: true,
Contexts: []string{"continuous-integration"},
Checks: []*RequiredStatusCheck{
Checks: []RequiredStatusCheck{
{
Context: "continuous-integration",
},
Expand Down Expand Up @@ -1186,7 +1186,7 @@ func TestRepositoriesService_GetBranchProtection_noDismissalRestrictions(t *test
RequiredStatusChecks: &RequiredStatusChecks{
Strict: true,
Contexts: []string{"continuous-integration"},
Checks: []*RequiredStatusCheck{
Checks: []RequiredStatusCheck{
{
Context: "continuous-integration",
},
Expand Down Expand Up @@ -1319,7 +1319,7 @@ func TestRepositoriesService_UpdateBranchProtection_Contexts(t *testing.T) {
RequiredStatusChecks: &RequiredStatusChecks{
Strict: true,
Contexts: []string{"continuous-integration"},
Checks: []*RequiredStatusCheck{
Checks: []RequiredStatusCheck{
{
Context: "continuous-integration",
},
Expand Down Expand Up @@ -1375,7 +1375,7 @@ func TestRepositoriesService_UpdateBranchProtection_Checks(t *testing.T) {
input := &ProtectionRequest{
RequiredStatusChecks: &RequiredStatusChecks{
Strict: true,
Checks: []*RequiredStatusCheck{
Checks: []RequiredStatusCheck{
{
Context: "continuous-integration",
},
Expand Down Expand Up @@ -1449,7 +1449,7 @@ func TestRepositoriesService_UpdateBranchProtection_Checks(t *testing.T) {
RequiredStatusChecks: &RequiredStatusChecks{
Strict: true,
Contexts: []string{"continuous-integration"},
Checks: []*RequiredStatusCheck{
Checks: []RequiredStatusCheck{
{
Context: "continuous-integration",
},
Expand Down Expand Up @@ -1603,7 +1603,7 @@ func TestRepositoriesService_GetRequiredStatusChecks(t *testing.T) {
want := &RequiredStatusChecks{
Strict: true,
Contexts: []string{"x", "y", "z"},
Checks: []*RequiredStatusCheck{
Checks: []RequiredStatusCheck{
{
Context: "x",
},
Expand Down Expand Up @@ -1699,7 +1699,7 @@ func TestRepositoriesService_UpdateRequiredStatusChecks_Contexts(t *testing.T) {
want := &RequiredStatusChecks{
Strict: true,
Contexts: []string{"continuous-integration"},
Checks: []*RequiredStatusCheck{
Checks: []RequiredStatusCheck{
{
Context: "continuous-integration",
},
Expand Down Expand Up @@ -1785,7 +1785,7 @@ func TestRepositoriesService_UpdateRequiredStatusChecks_Checks(t *testing.T) {
want := &RequiredStatusChecks{
Strict: true,
Contexts: []string{"continuous-integration"},
Checks: []*RequiredStatusCheck{
Checks: []RequiredStatusCheck{
{
Context: "continuous-integration",
},
Expand Down

0 comments on commit 1002c13

Please sign in to comment.