From 1002c13f340b27264cd11d5d651ed815bd000b64 Mon Sep 17 00:00:00 2001 From: Kevin Zhao Date: Fri, 28 Jan 2022 21:34:05 -0800 Subject: [PATCH] Use []RequiredStatusCheck instead of []*RequiredStatusCheck --- github/repos.go | 2 +- github/repos_test.go | 16 ++++++++-------- 2 files changed, 9 insertions(+), 9 deletions(-) diff --git a/github/repos.go b/github/repos.go index 228e8d869dd..608c590401f 100644 --- a/github/repos.go +++ b/github/repos.go @@ -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. diff --git a/github/repos_test.go b/github/repos_test.go index 7fa1e133176..96370706b91 100644 --- a/github/repos_test.go +++ b/github/repos_test.go @@ -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", }, @@ -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", }, @@ -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", }, @@ -1375,7 +1375,7 @@ func TestRepositoriesService_UpdateBranchProtection_Checks(t *testing.T) { input := &ProtectionRequest{ RequiredStatusChecks: &RequiredStatusChecks{ Strict: true, - Checks: []*RequiredStatusCheck{ + Checks: []RequiredStatusCheck{ { Context: "continuous-integration", }, @@ -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", }, @@ -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", }, @@ -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", }, @@ -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", },