return nil resource on error

this change also includes some additional minor code cleanup

fixes #58

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
 }

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
 }

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
 }

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
 }
 

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
 }

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
 }

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
 }

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
 }
 

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
 }
 

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
 }

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
 }

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
 }
 

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
 }
 

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
 }

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
 }
 

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
 }

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)
 }

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)
 }

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
 }

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
 }

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
 }
 

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
 }
 

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
 }

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)
 }

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
 }

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
 }
 

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
 }
 

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
 }