Skip to content
This repository has been archived by the owner on Jun 20, 2022. It is now read-only.

Updates the Issues API to use the new pattern #98

Merged
merged 5 commits into from
Apr 28, 2015
Merged

Conversation

dhruvsinghal
Copy link

@pengwynn please take a look.

@dhruvsinghal dhruvsinghal self-assigned this Mar 23, 2015
@pengwynn
Copy link
Collaborator

Awesome, @dhruvsinghal. Mind tackling #72 as part of this PR?

@dhruvsinghal
Copy link
Author

@pengwynn #72 changes are in.

}
url, err := uri.Expand(uriParams)
if err != nil {
return make([]Issue, 0), &Result{Err: err}
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Any reason to return an empty array here rather than nil?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

My bad: I didn't realize that you could return nil in place of slices.

@dhruvsinghal
Copy link
Author

@pengwynn Please take a look.

@feiranchen
Copy link
Contributor

@dhruvsinghal ^note on the comment above

@dhruvsinghal
Copy link
Author

@pengwynn Please take yet another look.

@pengwynn
Copy link
Collaborator

@dhruvsinghal Looks good. We'll merge after we figure out what's up with CI.

pengwynn added a commit that referenced this pull request Apr 28, 2015
Updates the Issues API to use the new pattern
@pengwynn pengwynn merged commit 3ee5bf3 into master Apr 28, 2015
@pengwynn pengwynn deleted the issues-refactoring branch April 28, 2015 18:10
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants