chore: check license headers

Author: ldez

.golangci.yml

@@ -11,12 +11,13 @@ linters:
     - gocritic
     - godot
     - gofmt
+    - goheader
     - goimports
     - gosec
     - misspell
     - nakedret
-    - perfsprint
     - paralleltest
+    - perfsprint
     - revive
     - sliceofpointers
     - stylecheck
@@ -35,6 +36,15 @@ linters-settings:
     enabled-checks:
       - commentedOutCode
       - commentFormatting
+  goheader:
+    values:
+      regexp:
+        CopyrightDate: "Copyright \\d{4} "
+    template: |-
+      {{CopyrightDate}}The go-github AUTHORS. All rights reserved.
+      
+      Use of this source code is governed by a BSD-style
+      license that can be found in the LICENSE file.
   gosec:
     excludes:
       # duplicates errcheck

github/admin_stats_test.go

@@ -1,3 +1,8 @@
+// Copyright 2017 The go-github AUTHORS. All rights reserved.
+//
+// Use of this source code is governed by a BSD-style
+// license that can be found in the LICENSE file.
+
 package github
 
 import (

github/enterprise_audit_log_test.go

@@ -1,6 +1,6 @@
 // Copyright 2021 The go-github AUTHORS. All rights reserved.
 //
-// `Use` of this source code is governed by a BSD-style
+// Use of this source code is governed by a BSD-style
 // license that can be found in the LICENSE file.
 
 package github

github/gists.go

@@ -1,6 +1,6 @@
 // Copyright 2013 The go-github AUTHORS. All rights reserved.
 //
-// Use of this source code is governed by BSD-style
+// Use of this source code is governed by a BSD-style
 // license that can be found in the LICENSE file.
 
 package github

github/orgs_audit_log_test.go

@@ -1,6 +1,6 @@
 // Copyright 2021 The go-github AUTHORS. All rights reserved.
 //
-// `Use` of this source code is governed by a BSD-style
+// Use of this source code is governed by a BSD-style
 // license that can be found in the LICENSE file.
 
 package github

github/pulls_reviewers.go

@@ -85,11 +85,7 @@ func (s *PullRequestsService) ListReviewers(ctx context.Context, owner, repo str
 func (s *PullRequestsService) RemoveReviewers(ctx context.Context, owner, repo string, number int, reviewers ReviewersRequest) (*Response, error) {
 	// reviewers.Reviewers may be empty if the caller wants to remove teams, but not users. Unlike AddReviewers,
 	// "reviewers" is a required param here. Reference: https://github.com/google/go-github/issues/3336
-	removeRequest := removeReviewersRequest{
-		NodeID:        reviewers.NodeID,
-		Reviewers:     reviewers.Reviewers,
-		TeamReviewers: reviewers.TeamReviewers,
-	}
+	removeRequest := removeReviewersRequest(reviewers)
 
 	if removeRequest.Reviewers == nil {
 		// GitHub accepts the empty list, but rejects null. Removing `omitempty` is not enough - we also have to promote nil to [].