Replace all helpers with github.Ptr

Author: alexandear

README.md

@@ -279,8 +279,8 @@ bool, and int values. For example:
 ```go
 // create a new private repository named "foo"
 repo := &github.Repository{
-Name: github.String("foo"),
-Private: github.Bool(true),
+Name: github.Ptr("foo"),
+Private: github.Ptr(true),
 }
 client.Repositories.Create(ctx, "", repo)
 ```

example/actionpermissions/main.go

@@ -44,7 +44,7 @@ func main() {
 
 fmt.Printf("Current ActionsPermissions %s\n", actionsPermissionsRepository.String())
 
-actionsPermissionsRepository = &github.ActionsPermissionsRepository{Enabled: github.Bool(true), AllowedActions: github.String("selected")}
+actionsPermissionsRepository = &github.ActionsPermissionsRepository{Enabled: github.Ptr(true), AllowedActions: github.Ptr("selected")}
 _, _, err = client.Repositories.EditActionsPermissions(ctx, *owner, *name, *actionsPermissionsRepository)
 if err != nil {
 log.Fatal(err)
@@ -59,15 +59,15 @@ func main() {
 
 fmt.Printf("Current ActionsAllowed %s\n", actionsAllowed.String())
 
-actionsAllowed = &github.ActionsAllowed{GithubOwnedAllowed: github.Bool(true), VerifiedAllowed: github.Bool(false), PatternsAllowed: []string{"a/b"}}
+actionsAllowed = &github.ActionsAllowed{GithubOwnedAllowed: github.Ptr(true), VerifiedAllowed: github.Ptr(false), PatternsAllowed: []string{"a/b"}}
 _, _, err = client.Repositories.EditActionsAllowed(ctx, *owner, *name, *actionsAllowed)
 if err != nil {
 log.Fatal(err)
 }
 
 fmt.Printf("Current ActionsAllowed %s\n", actionsAllowed.String())
 
-actionsPermissionsRepository = &github.ActionsPermissionsRepository{Enabled: github.Bool(true), AllowedActions: github.String("all")}
+actionsPermissionsRepository = &github.ActionsPermissionsRepository{Enabled: github.Ptr(true), AllowedActions: github.Ptr("all")}
 _, _, err = client.Repositories.EditActionsPermissions(ctx, *owner, *name, *actionsPermissionsRepository)
 if err != nil {
 log.Fatal(err)

example/commitpr/main.go

@@ -81,7 +81,7 @@ func getRef() (ref *github.Reference, err error) {
 if baseRef, _, err = client.Git.GetRef(ctx, *sourceOwner, *sourceRepo, "refs/heads/"+*baseBranch); err != nil {
 return nil, err
 }
-newRef := &github.Reference{Ref: github.String("refs/heads/" + *commitBranch), Object: &github.GitObject{SHA: baseRef.Object.SHA}}
+newRef := &github.Reference{Ref: github.Ptr("refs/heads/" + *commitBranch), Object: &github.GitObject{SHA: baseRef.Object.SHA}}
 ref, _, err = client.Git.CreateRef(ctx, *sourceOwner, *sourceRepo, newRef)
 return ref, err
 }
@@ -98,7 +98,7 @@ func getTree(ref *github.Reference) (tree *github.Tree, err error) {
 if err != nil {
 return nil, err
 }
-entries = append(entries, &github.TreeEntry{Path: github.String(file), Type: github.String("blob"), Content: github.String(string(content)), Mode: github.String("100644")})
+entries = append(entries, &github.TreeEntry{Path: github.Ptr(file), Type: github.Ptr("blob"), Content: github.Ptr(string(content)), Mode: github.Ptr("100644")})
 }
 
 tree, _, err = client.Git.CreateTree(ctx, *sourceOwner, *sourceRepo, *ref.Object.SHA, entries)
@@ -190,7 +190,7 @@ func createPR() (err error) {
 HeadRepo: repoBranch,
 Base: prBranch,
 Body: prDescription,
-MaintainerCanModify: github.Bool(true),
+MaintainerCanModify: github.Ptr(true),
 }
 
 pr, _, err := client.PullRequests.Create(ctx, *prRepoOwner, *prRepo, newPR)

example/newfilewithappauth/main.go

@@ -79,7 +79,7 @@ func main() {
 "example/foo.txt",
 &github.RepositoryContentFileOptions{
 Content: []byte("foo"),
-Message: github.String("sample commit"),
+Message: github.Ptr("sample commit"),
 SHA: nil,
 })
 if err != nil {

github/actions_artifacts_test.go

@@ -32,7 +32,7 @@ func TestActionsService_ListArtifacts(t *testing.T) {
 })
 
 opts := &ListArtifactsOptions{
-Name: String("TheArtifact"),
+Name: Ptr("TheArtifact"),
 ListOptions: ListOptions{Page: 2},
 }
 ctx := context.Background()
@@ -41,7 +41,7 @@ func TestActionsService_ListArtifacts(t *testing.T) {
 t.Errorf("Actions.ListArtifacts returned error: %v", err)
 }
 
-want := &ArtifactList{TotalCount: Int64(1), Artifacts: []*Artifact{{ID: Int64(1)}}}
+want := &ArtifactList{TotalCount: Ptr(1), Artifacts: []*Artifact{{ID: Ptr(1)}}}
 if !cmp.Equal(artifacts, want) {
 t.Errorf("Actions.ListArtifacts returned %+v, want %+v", artifacts, want)
 }
@@ -123,7 +123,7 @@ func TestActionsService_ListWorkflowRunArtifacts(t *testing.T) {
 t.Errorf("Actions.ListWorkflowRunArtifacts returned error: %v", err)
 }
 
-want := &ArtifactList{TotalCount: Int64(1), Artifacts: []*Artifact{{ID: Int64(1)}}}
+want := &ArtifactList{TotalCount: Ptr(1), Artifacts: []*Artifact{{ID: Ptr(1)}}}
 if !cmp.Equal(artifacts, want) {
 t.Errorf("Actions.ListWorkflowRunArtifacts returned %+v, want %+v", artifacts, want)
 }
@@ -205,11 +205,11 @@ func TestActionsService_GetArtifact(t *testing.T) {
 }
 
 want := &Artifact{
-ID: Int64(1),
-NodeID: String("xyz"),
-Name: String("a"),
-SizeInBytes: Int64(5),
-ArchiveDownloadURL: String("u"),
+ID: Ptr(1),
+NodeID: Ptr("xyz"),
+Name: Ptr("a"),
+SizeInBytes: Ptr(5),
+ArchiveDownloadURL: Ptr("u"),
 }
 if !cmp.Equal(artifact, want) {
 t.Errorf("Actions.GetArtifact returned %+v, want %+v", artifact, want)
@@ -438,22 +438,22 @@ func TestArtifact_Marshal(t *testing.T) {
 testJSONMarshal(t, &Artifact{}, "{}")
 
 u := &Artifact{
-ID: Int64(1),
-NodeID: String("nid"),
-Name: String("n"),
-SizeInBytes: Int64(1),
-URL: String("u"),
-ArchiveDownloadURL: String("a"),
-Expired: Bool(false),
+ID: Ptr(1),
+NodeID: Ptr("nid"),
+Name: Ptr("n"),
+SizeInBytes: Ptr(1),
+URL: Ptr("u"),
+ArchiveDownloadURL: Ptr("a"),
+Expired: Ptr(false),
 CreatedAt: &Timestamp{referenceTime},
 UpdatedAt: &Timestamp{referenceTime},
 ExpiresAt: &Timestamp{referenceTime},
 WorkflowRun: &ArtifactWorkflowRun{
-ID: Int64(1),
-RepositoryID: Int64(1),
-HeadRepositoryID: Int64(1),
-HeadBranch: String("b"),
-HeadSHA: String("s"),
+ID: Ptr(1),
+RepositoryID: Ptr(1),
+HeadRepositoryID: Ptr(1),
+HeadBranch: Ptr("b"),
+HeadSHA: Ptr("s"),
 },
 }
 
@@ -485,25 +485,25 @@ func TestArtifactList_Marshal(t *testing.T) {
 testJSONMarshal(t, &ArtifactList{}, "{}")
 
 u := &ArtifactList{
-TotalCount: Int64(1),
+TotalCount: Ptr(1),
 Artifacts: []*Artifact{
 {
-ID: Int64(1),
-NodeID: String("nid"),
-Name: String("n"),
-SizeInBytes: Int64(1),
-URL: String("u"),
-ArchiveDownloadURL: String("a"),
-Expired: Bool(false),
+ID: Ptr(1),
+NodeID: Ptr("nid"),
+Name: Ptr("n"),
+SizeInBytes: Ptr(1),
+URL: Ptr("u"),
+ArchiveDownloadURL: Ptr("a"),
+Expired: Ptr(false),
 CreatedAt: &Timestamp{referenceTime},
 UpdatedAt: &Timestamp{referenceTime},
 ExpiresAt: &Timestamp{referenceTime},
 WorkflowRun: &ArtifactWorkflowRun{
-ID: Int64(1),
-RepositoryID: Int64(1),
-HeadRepositoryID: Int64(1),
-HeadBranch: String("b"),
-HeadSHA: String("s"),
+ID: Ptr(1),
+RepositoryID: Ptr(1),
+HeadRepositoryID: Ptr(1),
+HeadBranch: Ptr("b"),
+HeadSHA: Ptr("s"),
 },
 },
 },

github/actions_cache_test.go

@@ -36,7 +36,7 @@ func TestActionsService_ListCaches(t *testing.T) {
 t.Errorf("Actions.ListCaches returned error: %v", err)
 }
 
-want := &ActionsCacheList{TotalCount: 1, ActionsCaches: []*ActionsCache{{ID: Int64(1)}}}
+want := &ActionsCacheList{TotalCount: 1, ActionsCaches: []*ActionsCache{{ID: Ptr(1)}}}
 if !cmp.Equal(cacheList, want) {
 t.Errorf("Actions.ListCaches returned %+v, want %+v", cacheList, want)
 }
@@ -106,19 +106,19 @@ func TestActionsService_DeleteCachesByKey(t *testing.T) {
 })
 
 ctx := context.Background()
-_, err := client.Actions.DeleteCachesByKey(ctx, "o", "r", "1", String("main"))
+_, err := client.Actions.DeleteCachesByKey(ctx, "o", "r", "1", Ptr("main"))
 if err != nil {
 t.Errorf("Actions.DeleteCachesByKey return error: %v", err)
 }
 
 const methodName = "DeleteCachesByKey"
 testBadOptions(t, methodName, func() (err error) {
-_, err = client.Actions.DeleteCachesByKey(ctx, "\n", "\n", "\n", String("\n"))
+_, err = client.Actions.DeleteCachesByKey(ctx, "\n", "\n", "\n", Ptr("\n"))
 return err
 })
 
 testNewRequestAndDoFailure(t, methodName, client, func() (*Response, error) {
-return client.Actions.DeleteCachesByKey(ctx, "o", "r", "1", String("main"))
+return client.Actions.DeleteCachesByKey(ctx, "o", "r", "1", Ptr("main"))
 })
 }
 
@@ -127,7 +127,7 @@ func TestActionsService_DeleteCachesByKey_invalidOwner(t *testing.T) {
 client, _, _ := setup(t)
 
 ctx := context.Background()
-_, err := client.Actions.DeleteCachesByKey(ctx, "%", "r", "1", String("main"))
+_, err := client.Actions.DeleteCachesByKey(ctx, "%", "r", "1", Ptr("main"))
 testURLParseError(t, err)
 }
 
@@ -136,7 +136,7 @@ func TestActionsService_DeleteCachesByKey_invalidRepo(t *testing.T) {
 client, _, _ := setup(t)
 
 ctx := context.Background()
-_, err := client.Actions.DeleteCachesByKey(ctx, "o", "%", "1", String("main"))
+_, err := client.Actions.DeleteCachesByKey(ctx, "o", "%", "1", Ptr("main"))
 testURLParseError(t, err)
 }
 func TestActionsService_DeleteCachesByKey_notFound(t *testing.T) {
@@ -149,7 +149,7 @@ func TestActionsService_DeleteCachesByKey_notFound(t *testing.T) {
 })
 
 ctx := context.Background()
-resp, err := client.Actions.DeleteCachesByKey(ctx, "o", "r", "1", String("main"))
+resp, err := client.Actions.DeleteCachesByKey(ctx, "o", "r", "1", Ptr("main"))
 if err == nil {
 t.Errorf("Expected HTTP 404 response")
 }
@@ -521,13 +521,13 @@ func TestActionsCache_Marshal(t *testing.T) {
 testJSONMarshal(t, &ActionsCache{}, "{}")
 
 u := &ActionsCache{
-ID: Int64(1),
-Ref: String("refAction"),
-Key: String("key1"),
-Version: String("alpha"),
+ID: Ptr(1),
+Ref: Ptr("refAction"),
+Key: Ptr("key1"),
+Version: Ptr("alpha"),
 LastAccessedAt: &Timestamp{referenceTime},
 CreatedAt: &Timestamp{referenceTime},
-SizeInBytes: Int64(1),
+SizeInBytes: Ptr(1),
 }
 
 want := `{
@@ -551,19 +551,19 @@ func TestActionsCacheList_Marshal(t *testing.T) {
 TotalCount: 2,
 ActionsCaches: []*ActionsCache{
 {
-ID: Int64(1),
-Key: String("key1"),
-Version: String("alpha"),
+ID: Ptr(1),
+Key: Ptr("key1"),
+Version: Ptr("alpha"),
 LastAccessedAt: &Timestamp{referenceTime},
 CreatedAt: &Timestamp{referenceTime},
-SizeInBytes: Int64(1),
+SizeInBytes: Ptr(1),
 },
 {
-ID: Int64(2),
-Ref: String("refAction"),
+ID: Ptr(2),
+Ref: Ptr("refAction"),
 LastAccessedAt: &Timestamp{referenceTime},
 CreatedAt: &Timestamp{referenceTime},
-SizeInBytes: Int64(1),
+SizeInBytes: Ptr(1),
 },
 },
 }

github/actions_oidc_test.go

@@ -64,7 +64,7 @@ func TestActionsService_GetRepoOIDCSubjectClaimCustomTemplate(t *testing.T) {
 t.Errorf("Actions.GetRepoOIDCSubjectClaimCustomTemplate returned error: %v", err)
 }
 
-want := &OIDCSubjectClaimCustomTemplate{UseDefault: Bool(false), IncludeClaimKeys: []string{"repo", "context"}}
+want := &OIDCSubjectClaimCustomTemplate{UseDefault: Ptr(false), IncludeClaimKeys: []string{"repo", "context"}}
 if !cmp.Equal(template, want) {
 t.Errorf("Actions.GetOrgOIDCSubjectClaimCustomTemplate returned %+v, want %+v", template, want)
 }
@@ -128,7 +128,7 @@ func TestActionsService_SetRepoOIDCSubjectClaimCustomTemplate(t *testing.T) {
 })
 
 input := &OIDCSubjectClaimCustomTemplate{
-UseDefault: Bool(false),
+UseDefault: Ptr(false),
 IncludeClaimKeys: []string{"repo", "context"},
 }
 ctx := context.Background()
@@ -161,7 +161,7 @@ func TestActionService_SetRepoOIDCSubjectClaimCustomTemplateToDefault(t *testing
 })
 
 input := &OIDCSubjectClaimCustomTemplate{
-UseDefault: Bool(true),
+UseDefault: Ptr(true),
 }
 ctx := context.Background()
 _, err := client.Actions.SetRepoOIDCSubjectClaimCustomTemplate(ctx, "o", "r", input)
@@ -185,7 +185,7 @@ func TestOIDCSubjectClaimCustomTemplate_Marshal(t *testing.T) {
 testJSONMarshal(t, &OIDCSubjectClaimCustomTemplate{}, "{}")
 
 u := &OIDCSubjectClaimCustomTemplate{
-UseDefault: Bool(false),
+UseDefault: Ptr(false),
 IncludeClaimKeys: []string{"s"},
 }
 

github/actions_permissions_enterprise.go

@@ -25,7 +25,7 @@ type ActionsPermissionsEnterprise struct {
 SelectedActionsURL *string `json:"selected_actions_url,omitempty"`
 }
 
-func (a ActionsPermissionsEnterprise) String() string {
+func (a ActionsPermissionsEnterprise) Ptr() string {
 return Stringify(a)
 }