diff --git a/github/activity_events.go b/github/activity_events.go index 7438a9a..3b6a260 100644 --- a/github/activity_events.go +++ b/github/activity_events.go @@ -89,6 +89,10 @@ func (s *ActivityService) ListEvents(opt *ListOptions) ([]Event, *Response, erro events := new([]Event) resp, err := s.client.Do(req, events) + if err != nil { + return nil, resp, err + } + return *events, resp, err } @@ -111,6 +115,10 @@ func (s *ActivityService) ListRepositoryEvents(owner, repo string, opt *ListOpti events := new([]Event) resp, err := s.client.Do(req, events) + if err != nil { + return nil, resp, err + } + return *events, resp, err } @@ -133,6 +141,10 @@ func (s *ActivityService) ListIssueEventsForRepository(owner, repo string, opt * events := new([]Event) resp, err := s.client.Do(req, events) + if err != nil { + return nil, resp, err + } + return *events, resp, err } @@ -155,6 +167,10 @@ func (s *ActivityService) ListEventsForRepoNetwork(owner, repo string, opt *List events := new([]Event) resp, err := s.client.Do(req, events) + if err != nil { + return nil, resp, err + } + return *events, resp, err } @@ -177,6 +193,10 @@ func (s *ActivityService) ListEventsForOrganization(org string, opt *ListOptions events := new([]Event) resp, err := s.client.Do(req, events) + if err != nil { + return nil, resp, err + } + return *events, resp, err } @@ -206,6 +226,10 @@ func (s *ActivityService) ListEventsPerformedByUser(user string, publicOnly bool events := new([]Event) resp, err := s.client.Do(req, events) + if err != nil { + return nil, resp, err + } + return *events, resp, err } @@ -235,6 +259,10 @@ func (s *ActivityService) ListEventsRecievedByUser(user string, publicOnly bool, events := new([]Event) resp, err := s.client.Do(req, events) + if err != nil { + return nil, resp, err + } + return *events, resp, err } @@ -257,5 +285,9 @@ func (s *ActivityService) ListUserEventsForOrganization(org, user string, opt *L events := new([]Event) resp, err := s.client.Do(req, events) + if err != nil { + return nil, resp, err + } + return *events, resp, err } diff --git a/github/activity_star.go b/github/activity_star.go index 95232fc..6192ee7 100644 --- a/github/activity_star.go +++ b/github/activity_star.go @@ -52,5 +52,9 @@ func (s *ActivityService) ListStarred(user string, opt *ActivityListStarredOptio repos := new([]Repository) resp, err := s.client.Do(req, repos) + if err != nil { + return nil, resp, err + } + return *repos, resp, err } diff --git a/github/gists.go b/github/gists.go index 65e7710..c7e3f47 100644 --- a/github/gists.go +++ b/github/gists.go @@ -87,6 +87,10 @@ func (s *GistsService) List(user string, opt *GistListOptions) ([]Gist, *Respons gists := new([]Gist) resp, err := s.client.Do(req, gists) + if err != nil { + return nil, resp, err + } + return *gists, resp, err } @@ -110,6 +114,10 @@ func (s *GistsService) ListAll(opt *GistListOptions) ([]Gist, *Response, error) gists := new([]Gist) resp, err := s.client.Do(req, gists) + if err != nil { + return nil, resp, err + } + return *gists, resp, err } @@ -133,6 +141,10 @@ func (s *GistsService) ListStarred(opt *GistListOptions) ([]Gist, *Response, err gists := new([]Gist) resp, err := s.client.Do(req, gists) + if err != nil { + return nil, resp, err + } + return *gists, resp, err } @@ -147,6 +159,10 @@ func (s *GistsService) Get(id string) (*Gist, *Response, error) { } gist := new(Gist) resp, err := s.client.Do(req, gist) + if err != nil { + return nil, resp, err + } + return gist, resp, err } @@ -161,6 +177,10 @@ func (s *GistsService) Create(gist *Gist) (*Gist, *Response, error) { } g := new(Gist) resp, err := s.client.Do(req, g) + if err != nil { + return nil, resp, err + } + return g, resp, err } @@ -175,6 +195,10 @@ func (s *GistsService) Edit(id string, gist *Gist) (*Gist, *Response, error) { } g := new(Gist) resp, err := s.client.Do(req, g) + if err != nil { + return nil, resp, err + } + return g, resp, err } @@ -237,7 +261,12 @@ func (s *GistsService) Fork(id string) (*Gist, *Response, error) { if err != nil { return nil, nil, err } + g := new(Gist) resp, err := s.client.Do(req, g) + if err != nil { + return nil, resp, err + } + return g, resp, err } diff --git a/github/gists_comments.go b/github/gists_comments.go index dabd68a..e700bbd 100644 --- a/github/gists_comments.go +++ b/github/gists_comments.go @@ -35,6 +35,10 @@ func (s *GistsService) ListComments(gistID string) ([]GistComment, *Response, er comments := new([]GistComment) resp, err := s.client.Do(req, comments) + if err != nil { + return nil, resp, err + } + return *comments, resp, err } @@ -50,6 +54,10 @@ func (s *GistsService) GetComment(gistID string, commentID int) (*GistComment, * c := new(GistComment) resp, err := s.client.Do(req, c) + if err != nil { + return nil, resp, err + } + return c, resp, err } @@ -65,6 +73,10 @@ func (s *GistsService) CreateComment(gistID string, comment *GistComment) (*Gist c := new(GistComment) resp, err := s.client.Do(req, c) + if err != nil { + return nil, resp, err + } + return c, resp, err } @@ -80,6 +92,10 @@ func (s *GistsService) EditComment(gistID string, commentID int, comment *GistCo c := new(GistComment) resp, err := s.client.Do(req, c) + if err != nil { + return nil, resp, err + } + return c, resp, err } diff --git a/github/git_commits.go b/github/git_commits.go index 0a39576..6e73420 100644 --- a/github/git_commits.go +++ b/github/git_commits.go @@ -49,6 +49,10 @@ func (s *GitService) GetCommit(owner string, repo string, sha string) (*Commit, c := new(Commit) resp, err := s.client.Do(req, c) + if err != nil { + return nil, resp, err + } + return c, resp, err } @@ -68,5 +72,9 @@ func (s *GitService) CreateCommit(owner string, repo string, commit *Commit) (*C c := new(Commit) resp, err := s.client.Do(req, c) + if err != nil { + return nil, resp, err + } + return c, resp, err } diff --git a/github/git_trees.go b/github/git_trees.go index 92af126..5d06a81 100644 --- a/github/git_trees.go +++ b/github/git_trees.go @@ -48,6 +48,10 @@ func (s *GitService) GetTree(owner string, repo string, sha string, recursive bo t := new(Tree) resp, err := s.client.Do(req, t) + if err != nil { + return nil, resp, err + } + return t, resp, err } @@ -76,5 +80,9 @@ func (s *GitService) CreateTree(owner string, repo string, baseTree string, entr t := new(Tree) resp, err := s.client.Do(req, t) + if err != nil { + return nil, resp, err + } + return t, resp, err } diff --git a/github/issues.go b/github/issues.go index 2c8a148..d810283 100644 --- a/github/issues.go +++ b/github/issues.go @@ -119,6 +119,10 @@ func (s *IssuesService) listIssues(u string, opt *IssueListOptions) ([]Issue, *R issues := new([]Issue) resp, err := s.client.Do(req, issues) + if err != nil { + return nil, resp, err + } + return *issues, resp, err } @@ -178,6 +182,10 @@ func (s *IssuesService) ListByRepo(owner string, repo string, opt *IssueListByRe issues := new([]Issue) resp, err := s.client.Do(req, issues) + if err != nil { + return nil, resp, err + } + return *issues, resp, err } @@ -192,6 +200,10 @@ func (s *IssuesService) Get(owner string, repo string, number int) (*Issue, *Res } issue := new(Issue) resp, err := s.client.Do(req, issue) + if err != nil { + return nil, resp, err + } + return issue, resp, err } @@ -206,6 +218,10 @@ func (s *IssuesService) Create(owner string, repo string, issue *Issue) (*Issue, } i := new(Issue) resp, err := s.client.Do(req, i) + if err != nil { + return nil, resp, err + } + return i, resp, err } @@ -220,5 +236,9 @@ func (s *IssuesService) Edit(owner string, repo string, number int, issue *Issue } i := new(Issue) resp, err := s.client.Do(req, i) + if err != nil { + return nil, resp, err + } + return i, resp, err } diff --git a/github/issues_assignees.go b/github/issues_assignees.go index bd88706..a46d866 100644 --- a/github/issues_assignees.go +++ b/github/issues_assignees.go @@ -19,6 +19,10 @@ func (s *IssuesService) ListAssignees(owner string, repo string) ([]User, *Respo } assignees := new([]User) resp, err := s.client.Do(req, assignees) + if err != nil { + return nil, resp, err + } + return *assignees, resp, err } diff --git a/github/issues_comments.go b/github/issues_comments.go index 222c141..d53db80 100644 --- a/github/issues_comments.go +++ b/github/issues_comments.go @@ -66,6 +66,10 @@ func (s *IssuesService) ListComments(owner string, repo string, number int, opt } comments := new([]IssueComment) resp, err := s.client.Do(req, comments) + if err != nil { + return nil, resp, err + } + return *comments, resp, err } @@ -81,6 +85,10 @@ func (s *IssuesService) GetComment(owner string, repo string, id int) (*IssueCom } comment := new(IssueComment) resp, err := s.client.Do(req, comment) + if err != nil { + return nil, resp, err + } + return comment, resp, err } @@ -95,6 +103,10 @@ func (s *IssuesService) CreateComment(owner string, repo string, number int, com } c := new(IssueComment) resp, err := s.client.Do(req, c) + if err != nil { + return nil, resp, err + } + return c, resp, err } @@ -109,6 +121,10 @@ func (s *IssuesService) EditComment(owner string, repo string, id int, comment * } c := new(IssueComment) resp, err := s.client.Do(req, c) + if err != nil { + return nil, resp, err + } + return c, resp, err } diff --git a/github/issues_labels.go b/github/issues_labels.go index fae8287..eb7b1b3 100644 --- a/github/issues_labels.go +++ b/github/issues_labels.go @@ -27,8 +27,13 @@ func (s *IssuesService) ListLabels(owner string, repo string) ([]Label, *Respons if err != nil { return nil, nil, err } + labels := new([]Label) resp, err := s.client.Do(req, labels) + if err != nil { + return nil, resp, err + } + return *labels, resp, err } @@ -41,8 +46,13 @@ func (s *IssuesService) GetLabel(owner string, repo string, name string) (*Label if err != nil { return nil, nil, err } + label := new(Label) resp, err := s.client.Do(req, label) + if err != nil { + return nil, resp, err + } + return label, resp, err } @@ -55,8 +65,13 @@ func (s *IssuesService) CreateLabel(owner string, repo string, label *Label) (*L if err != nil { return nil, nil, err } + l := new(Label) resp, err := s.client.Do(req, l) + if err != nil { + return nil, resp, err + } + return l, resp, err } @@ -69,8 +84,13 @@ func (s *IssuesService) EditLabel(owner string, repo string, name string, label if err != nil { return nil, nil, err } + l := new(Label) resp, err := s.client.Do(req, l) + if err != nil { + return nil, resp, err + } + return l, resp, err } @@ -95,8 +115,13 @@ func (s *IssuesService) ListLabelsByIssue(owner string, repo string, number int) if err != nil { return nil, nil, err } + labels := new([]Label) resp, err := s.client.Do(req, labels) + if err != nil { + return nil, resp, err + } + return *labels, resp, err } @@ -109,8 +134,13 @@ func (s *IssuesService) AddLabelsToIssue(owner string, repo string, number int, if err != nil { return nil, nil, err } + l := new([]Label) resp, err := s.client.Do(req, l) + if err != nil { + return nil, resp, err + } + return *l, resp, err } @@ -135,8 +165,13 @@ func (s *IssuesService) ReplaceLabelsForIssue(owner string, repo string, number if err != nil { return nil, nil, err } + l := new([]Label) resp, err := s.client.Do(req, l) + if err != nil { + return nil, resp, err + } + return *l, resp, err } @@ -161,7 +196,12 @@ func (s *IssuesService) ListLabelsForMilestone(owner string, repo string, number if err != nil { return nil, nil, err } + labels := new([]Label) resp, err := s.client.Do(req, labels) + if err != nil { + return nil, resp, err + } + return *labels, resp, err } diff --git a/github/orgs.go b/github/orgs.go index 5b27e28..ccbbd7a 100644 --- a/github/orgs.go +++ b/github/orgs.go @@ -86,6 +86,10 @@ func (s *OrganizationsService) List(user string, opt *ListOptions) ([]Organizati orgs := new([]Organization) resp, err := s.client.Do(req, orgs) + if err != nil { + return nil, resp, err + } + return *orgs, resp, err } @@ -101,6 +105,10 @@ func (s *OrganizationsService) Get(org string) (*Organization, *Response, error) organization := new(Organization) resp, err := s.client.Do(req, organization) + if err != nil { + return nil, resp, err + } + return organization, resp, err } @@ -116,5 +124,9 @@ func (s *OrganizationsService) Edit(name string, org *Organization) (*Organizati o := new(Organization) resp, err := s.client.Do(req, o) + if err != nil { + return nil, resp, err + } + return o, resp, err } diff --git a/github/orgs_members.go b/github/orgs_members.go index d50cf17..902dc5a 100644 --- a/github/orgs_members.go +++ b/github/orgs_members.go @@ -27,6 +27,10 @@ func (s *OrganizationsService) ListMembers(org string, publicOnly bool) ([]User, members := new([]User) resp, err := s.client.Do(req, members) + if err != nil { + return nil, resp, err + } + return *members, resp, err } diff --git a/github/orgs_teams.go b/github/orgs_teams.go index dda4cb2..46c434b 100644 --- a/github/orgs_teams.go +++ b/github/orgs_teams.go @@ -35,6 +35,10 @@ func (s *OrganizationsService) ListTeams(org string) ([]Team, *Response, error) teams := new([]Team) resp, err := s.client.Do(req, teams) + if err != nil { + return nil, resp, err + } + return *teams, resp, err } @@ -50,6 +54,10 @@ func (s *OrganizationsService) GetTeam(team int) (*Team, *Response, error) { t := new(Team) resp, err := s.client.Do(req, t) + if err != nil { + return nil, resp, err + } + return t, resp, err } @@ -65,6 +73,10 @@ func (s *OrganizationsService) CreateTeam(org string, team *Team) (*Team, *Respo t := new(Team) resp, err := s.client.Do(req, t) + if err != nil { + return nil, resp, err + } + return t, resp, err } @@ -80,6 +92,10 @@ func (s *OrganizationsService) EditTeam(id int, team *Team) (*Team, *Response, e t := new(Team) resp, err := s.client.Do(req, t) + if err != nil { + return nil, resp, err + } + return t, resp, err } @@ -109,6 +125,10 @@ func (s *OrganizationsService) ListTeamMembers(team int) ([]User, *Response, err members := new([]User) resp, err := s.client.Do(req, members) + if err != nil { + return nil, resp, err + } + return *members, resp, err } @@ -165,6 +185,10 @@ func (s *OrganizationsService) ListTeamRepos(team int) ([]Repository, *Response, repos := new([]Repository) resp, err := s.client.Do(req, repos) + if err != nil { + return nil, resp, err + } + return *repos, resp, err } diff --git a/github/pulls.go b/github/pulls.go index fac3d9c..1e6375e 100644 --- a/github/pulls.go +++ b/github/pulls.go @@ -82,6 +82,10 @@ func (s *PullRequestsService) List(owner string, repo string, opt *PullRequestLi pulls := new([]PullRequest) resp, err := s.client.Do(req, pulls) + if err != nil { + return nil, resp, err + } + return *pulls, resp, err } @@ -94,8 +98,13 @@ func (s *PullRequestsService) Get(owner string, repo string, number int) (*PullR if err != nil { return nil, nil, err } + pull := new(PullRequest) resp, err := s.client.Do(req, pull) + if err != nil { + return nil, resp, err + } + return pull, resp, err } @@ -108,8 +117,13 @@ func (s *PullRequestsService) Create(owner string, repo string, pull *PullReques if err != nil { return nil, nil, err } + p := new(PullRequest) resp, err := s.client.Do(req, p) + if err != nil { + return nil, resp, err + } + return p, resp, err } @@ -122,7 +136,12 @@ func (s *PullRequestsService) Edit(owner string, repo string, number int, pull * if err != nil { return nil, nil, err } + p := new(PullRequest) resp, err := s.client.Do(req, p) + if err != nil { + return nil, resp, err + } + return p, resp, err } diff --git a/github/pulls_comments.go b/github/pulls_comments.go index 7505d4b..269b19e 100644 --- a/github/pulls_comments.go +++ b/github/pulls_comments.go @@ -68,8 +68,13 @@ func (s *PullRequestsService) ListComments(owner string, repo string, number int if err != nil { return nil, nil, err } + comments := new([]PullRequestComment) resp, err := s.client.Do(req, comments) + if err != nil { + return nil, resp, err + } + return *comments, resp, err } @@ -82,8 +87,13 @@ func (s *PullRequestsService) GetComment(owner string, repo string, number int) if err != nil { return nil, nil, err } + comment := new(PullRequestComment) resp, err := s.client.Do(req, comment) + if err != nil { + return nil, resp, err + } + return comment, resp, err } @@ -96,8 +106,13 @@ func (s *PullRequestsService) CreateComment(owner string, repo string, number in if err != nil { return nil, nil, err } + c := new(PullRequestComment) resp, err := s.client.Do(req, c) + if err != nil { + return nil, resp, err + } + return c, resp, err } @@ -110,8 +125,13 @@ func (s *PullRequestsService) EditComment(owner string, repo string, number int, if err != nil { return nil, nil, err } + c := new(PullRequestComment) resp, err := s.client.Do(req, c) + if err != nil { + return nil, resp, err + } + return c, resp, err } diff --git a/github/repos.go b/github/repos.go index c5246c7..2f3ead0 100644 --- a/github/repos.go +++ b/github/repos.go @@ -91,6 +91,10 @@ func (s *RepositoriesService) List(user string, opt *RepositoryListOptions) ([]R repos := new([]Repository) resp, err := s.client.Do(req, repos) + if err != nil { + return nil, resp, err + } + return *repos, resp, err } @@ -121,6 +125,10 @@ func (s *RepositoriesService) ListByOrg(org string, opt *RepositoryListByOrgOpti repos := new([]Repository) resp, err := s.client.Do(req, repos) + if err != nil { + return nil, resp, err + } + return *repos, resp, err } @@ -149,6 +157,10 @@ func (s *RepositoriesService) ListAll(opt *RepositoryListAllOptions) ([]Reposito repos := new([]Repository) resp, err := s.client.Do(req, repos) + if err != nil { + return nil, resp, err + } + return *repos, resp, err } @@ -172,6 +184,10 @@ func (s *RepositoriesService) Create(org string, repo *Repository) (*Repository, r := new(Repository) resp, err := s.client.Do(req, r) + if err != nil { + return nil, resp, err + } + return r, resp, err } @@ -184,8 +200,13 @@ func (s *RepositoriesService) Get(owner, repo string) (*Repository, *Response, e if err != nil { return nil, nil, err } + repository := new(Repository) resp, err := s.client.Do(req, repository) + if err != nil { + return nil, resp, err + } + return repository, resp, err } @@ -198,8 +219,13 @@ func (s *RepositoriesService) Edit(owner, repo string, repository *Repository) ( if err != nil { return nil, nil, err } + r := new(Repository) resp, err := s.client.Do(req, r) + if err != nil { + return nil, resp, err + } + return r, resp, err } @@ -222,5 +248,9 @@ func (s *RepositoriesService) ListLanguages(owner string, repository string) (ma languages := make(map[string]int) resp, err := s.client.Do(req, &languages) + if err != nil { + return nil, resp, err + } + return languages, resp, err } diff --git a/github/repos_collaborators.go b/github/repos_collaborators.go index 1dee956..a84a92e 100644 --- a/github/repos_collaborators.go +++ b/github/repos_collaborators.go @@ -5,9 +5,7 @@ package github -import ( - "fmt" -) +import "fmt" // ListCollaborators lists the Github users that have access to the repository. // @@ -21,6 +19,10 @@ func (s *RepositoriesService) ListCollaborators(owner, repo string) ([]User, *Re users := new([]User) resp, err := s.client.Do(req, users) + if err != nil { + return nil, resp, err + } + return *users, resp, err } @@ -51,8 +53,7 @@ func (s *RepositoriesService) AddCollaborator(owner, repo, user string) (*Respon if err != nil { return nil, err } - resp, err := s.client.Do(req, nil) - return resp, err + return s.client.Do(req, nil) } // RemoveCollaborator removes the specified Github user as collaborator from the given repo. @@ -65,6 +66,5 @@ func (s *RepositoriesService) RemoveCollaborator(owner, repo, user string) (*Res if err != nil { return nil, err } - resp, err := s.client.Do(req, nil) - return resp, err + return s.client.Do(req, nil) } diff --git a/github/repos_comments.go b/github/repos_comments.go index 0348905..7b06cd9 100644 --- a/github/repos_comments.go +++ b/github/repos_comments.go @@ -40,8 +40,13 @@ func (s *RepositoriesService) ListComments(owner, repo string) ([]RepositoryComm if err != nil { return nil, nil, err } + comments := new([]RepositoryComment) resp, err := s.client.Do(req, comments) + if err != nil { + return nil, resp, err + } + return *comments, resp, err } @@ -54,8 +59,13 @@ func (s *RepositoriesService) ListCommitComments(owner, repo, sha string) ([]Rep if err != nil { return nil, nil, err } + comments := new([]RepositoryComment) resp, err := s.client.Do(req, comments) + if err != nil { + return nil, resp, err + } + return *comments, resp, err } @@ -69,8 +79,13 @@ func (s *RepositoriesService) CreateComment(owner, repo, sha string, comment *Re if err != nil { return nil, nil, err } + c := new(RepositoryComment) resp, err := s.client.Do(req, c) + if err != nil { + return nil, resp, err + } + return c, resp, err } @@ -83,8 +98,13 @@ func (s *RepositoriesService) GetComment(owner, repo string, id int) (*Repositor if err != nil { return nil, nil, err } + c := new(RepositoryComment) resp, err := s.client.Do(req, c) + if err != nil { + return nil, resp, err + } + return c, resp, err } @@ -97,8 +117,13 @@ func (s *RepositoriesService) UpdateComment(owner, repo string, id int, comment if err != nil { return nil, nil, err } + c := new(RepositoryComment) resp, err := s.client.Do(req, c) + if err != nil { + return nil, resp, err + } + return c, resp, err } @@ -111,6 +136,5 @@ func (s *RepositoriesService) DeleteComment(owner, repo string, id int) (*Respon if err != nil { return nil, err } - resp, err := s.client.Do(req, nil) - return resp, err + return s.client.Do(req, nil) } diff --git a/github/repos_commits.go b/github/repos_commits.go index b61631b..588cad4 100644 --- a/github/repos_commits.go +++ b/github/repos_commits.go @@ -122,6 +122,10 @@ func (s *RepositoriesService) ListCommits(owner, repo string, opts *CommitsListO commits := new([]RepositoryCommit) resp, err := s.client.Do(req, commits) + if err != nil { + return nil, resp, err + } + return *commits, resp, err } @@ -140,6 +144,10 @@ func (s *RepositoriesService) GetCommit(owner, repo, sha string) (*RepositoryCom commit := new(RepositoryCommit) resp, err := s.client.Do(req, commit) + if err != nil { + return nil, resp, err + } + return commit, resp, err } @@ -157,5 +165,9 @@ func (s *RepositoriesService) CompareCommits(owner, repo string, base, head stri comp := new(CommitsComparison) resp, err := s.client.Do(req, comp) + if err != nil { + return nil, resp, err + } + return comp, resp, err } diff --git a/github/repos_forks.go b/github/repos_forks.go index a0f9204..f72a2cd 100644 --- a/github/repos_forks.go +++ b/github/repos_forks.go @@ -37,6 +37,10 @@ func (s *RepositoriesService) ListForks(owner, repo string, opt *RepositoryListF repos := new([]Repository) resp, err := s.client.Do(req, repos) + if err != nil { + return nil, resp, err + } + return *repos, resp, err } @@ -66,5 +70,9 @@ func (s *RepositoriesService) CreateFork(owner, repo string, opt *RepositoryCrea fork := new(Repository) resp, err := s.client.Do(req, fork) + if err != nil { + return nil, resp, err + } + return fork, resp, err } diff --git a/github/repos_hooks.go b/github/repos_hooks.go index 44d20e1..e9c3fbb 100644 --- a/github/repos_hooks.go +++ b/github/repos_hooks.go @@ -92,8 +92,13 @@ func (s *RepositoriesService) CreateHook(owner, repo string, hook *Hook) (*Hook, if err != nil { return nil, nil, err } + h := new(Hook) resp, err := s.client.Do(req, h) + if err != nil { + return nil, resp, err + } + return h, resp, err } @@ -117,6 +122,10 @@ func (s *RepositoriesService) ListHooks(owner, repo string, opt *ListOptions) ([ hooks := new([]Hook) resp, err := s.client.Do(req, hooks) + if err != nil { + return nil, resp, err + } + return *hooks, resp, err } diff --git a/github/repos_keys.go b/github/repos_keys.go index f64fac5..abb1ca7 100644 --- a/github/repos_keys.go +++ b/github/repos_keys.go @@ -22,6 +22,10 @@ func (s *RepositoriesService) ListKeys(owner string, repo string) ([]Key, *Respo keys := new([]Key) resp, err := s.client.Do(req, keys) + if err != nil { + return nil, resp, err + } + return *keys, resp, err } @@ -38,6 +42,10 @@ func (s *RepositoriesService) GetKey(owner string, repo string, id int) (*Key, * key := new(Key) resp, err := s.client.Do(req, key) + if err != nil { + return nil, resp, err + } + return key, resp, err } @@ -54,6 +62,10 @@ func (s *RepositoriesService) CreateKey(owner string, repo string, key *Key) (*K k := new(Key) resp, err := s.client.Do(req, k) + if err != nil { + return nil, resp, err + } + return k, resp, err } @@ -70,6 +82,10 @@ func (s *RepositoriesService) EditKey(owner string, repo string, id int, key *Ke k := new(Key) resp, err := s.client.Do(req, k) + if err != nil { + return nil, resp, err + } + return k, resp, err } diff --git a/github/repos_statuses.go b/github/repos_statuses.go index 7d7c2a1..0ba3312 100644 --- a/github/repos_statuses.go +++ b/github/repos_statuses.go @@ -47,6 +47,10 @@ func (s *RepositoriesService) ListStatuses(owner, repo, ref string) ([]RepoStatu statuses := new([]RepoStatus) resp, err := s.client.Do(req, statuses) + if err != nil { + return nil, resp, err + } + return *statuses, resp, err } @@ -63,5 +67,9 @@ func (s *RepositoriesService) CreateStatus(owner, repo, ref string, status *Repo statuses := new(RepoStatus) resp, err := s.client.Do(req, statuses) + if err != nil { + return nil, resp, err + } + return statuses, resp, err } diff --git a/github/search.go b/github/search.go index 3bdbc6f..f1f3af6 100644 --- a/github/search.go +++ b/github/search.go @@ -133,6 +133,5 @@ func (s *SearchService) search(searchType string, query string, opt *SearchOptio } req.Header.Add("Accept", mimePreview) - resp, err := s.client.Do(req, result) - return resp, err + return s.client.Do(req, result) } diff --git a/github/users.go b/github/users.go index 1414aa5..aae5d44 100644 --- a/github/users.go +++ b/github/users.go @@ -60,6 +60,10 @@ func (s *UsersService) Get(user string) (*User, *Response, error) { uResp := new(User) resp, err := s.client.Do(req, uResp) + if err != nil { + return nil, resp, err + } + return uResp, resp, err } @@ -75,6 +79,10 @@ func (s *UsersService) Edit(user *User) (*User, *Response, error) { uResp := new(User) resp, err := s.client.Do(req, uResp) + if err != nil { + return nil, resp, err + } + return uResp, resp, err } @@ -101,5 +109,9 @@ func (s *UsersService) ListAll(opt *UserListOptions) ([]User, *Response, error) users := new([]User) resp, err := s.client.Do(req, users) + if err != nil { + return nil, resp, err + } + return *users, resp, err } diff --git a/github/users_emails.go b/github/users_emails.go index acf9b1c..891c32f 100644 --- a/github/users_emails.go +++ b/github/users_emails.go @@ -20,6 +20,10 @@ func (s *UsersService) ListEmails() ([]UserEmail, *Response, error) { emails := new([]UserEmail) resp, err := s.client.Do(req, emails) + if err != nil { + return nil, resp, err + } + return *emails, resp, err } @@ -35,6 +39,10 @@ func (s *UsersService) AddEmails(emails []UserEmail) ([]UserEmail, *Response, er e := new([]UserEmail) resp, err := s.client.Do(req, e) + if err != nil { + return nil, resp, err + } + return *e, resp, err } diff --git a/github/users_followers.go b/github/users_followers.go index a629769..e5fc692 100644 --- a/github/users_followers.go +++ b/github/users_followers.go @@ -28,6 +28,10 @@ func (s *UsersService) ListFollowers(user string) ([]User, *Response, error) { users := new([]User) resp, err := s.client.Do(req, users) + if err != nil { + return nil, resp, err + } + return *users, resp, err } @@ -57,6 +61,10 @@ func (s *UsersService) ListFollowing(user string, opt *ListOptions) ([]User, *Re users := new([]User) resp, err := s.client.Do(req, users) + if err != nil { + return nil, resp, err + } + return *users, resp, err } diff --git a/github/users_keys.go b/github/users_keys.go index 51d1483..7b4ad1e 100644 --- a/github/users_keys.go +++ b/github/users_keys.go @@ -38,6 +38,10 @@ func (s *UsersService) ListKeys(user string) ([]Key, *Response, error) { keys := new([]Key) resp, err := s.client.Do(req, keys) + if err != nil { + return nil, resp, err + } + return *keys, resp, err } @@ -54,6 +58,10 @@ func (s *UsersService) GetKey(id int) (*Key, *Response, error) { key := new(Key) resp, err := s.client.Do(req, key) + if err != nil { + return nil, resp, err + } + return key, resp, err } @@ -70,6 +78,10 @@ func (s *UsersService) CreateKey(key *Key) (*Key, *Response, error) { k := new(Key) resp, err := s.client.Do(req, k) + if err != nil { + return nil, resp, err + } + return k, resp, err } @@ -86,6 +98,10 @@ func (s *UsersService) EditKey(id int, key *Key) (*Key, *Response, error) { k := new(Key) resp, err := s.client.Do(req, k) + if err != nil { + return nil, resp, err + } + return k, resp, err }