Aligns other structs with the new convention

Required fields should be values, not pointers, because a lack of a value is an
invalid specification.

Author: alindeman

github/repos.go

@@ -523,12 +523,12 @@ type ProtectionRequest struct {
 // RequiredStatusChecks represents the protection status of a individual branch.
 type RequiredStatusChecks struct {
 	// Enforce required status checks for repository administrators.
-	IncludeAdmins *bool `json:"include_admins,omitempty"`
+	IncludeAdmins bool `json:"include_admins"`
 	// Require branches to be up to date before merging.
-	Strict *bool `json:"strict,omitempty"`
+	Strict bool `json:"strict"`
 	// The list of status checks to require in order to merge into this
 	// branch.
-	Contexts *[]string `json:"contexts,omitempty"`
+	Contexts []string `json:"contexts"`
 }
 
 // RequiredPullRequestReviews represents the protection configuration for pull requests.

github/repos_test.go

@@ -491,9 +491,9 @@ func TestRepositoriesService_GetBranchProtection(t *testing.T) {
 
 	want := &Protection{
 		RequiredStatusChecks: &RequiredStatusChecks{
-			IncludeAdmins: Bool(true),
-			Strict:        Bool(true),
-			Contexts:      &[]string{"continuous-integration"},
+			IncludeAdmins: true,
+			Strict:        true,
+			Contexts:      []string{"continuous-integration"},
 		},
 		RequiredPullRequestReviews: &RequiredPullRequestReviews{
 			IncludeAdmins: true,
@@ -518,9 +518,9 @@ func TestRepositoriesService_UpdateBranchProtection(t *testing.T) {
 
 	input := &ProtectionRequest{
 		RequiredStatusChecks: &RequiredStatusChecks{
-			IncludeAdmins: Bool(true),
-			Strict:        Bool(true),
-			Contexts:      &[]string{"continuous-integration"},
+			IncludeAdmins: true,
+			Strict:        true,
+			Contexts:      []string{"continuous-integration"},
 		},
 		RequiredPullRequestReviews: &RequiredPullRequestReviews{
 			IncludeAdmins: true,
@@ -550,9 +550,9 @@ func TestRepositoriesService_UpdateBranchProtection(t *testing.T) {
 
 	want := &Protection{
 		RequiredStatusChecks: &RequiredStatusChecks{
-			IncludeAdmins: Bool(true),
-			Strict:        Bool(true),
-			Contexts:      &[]string{"continuous-integration"},
+			IncludeAdmins: true,
+			Strict:        true,
+			Contexts:      []string{"continuous-integration"},
 		},
 		RequiredPullRequestReviews: &RequiredPullRequestReviews{
 			IncludeAdmins: true,