Skip to content

Return array of DataObjects if such is requested.#546

Merged
ycombinator merged 9 commits intorackspace:workingfrom
ycombinator:gh-545
Mar 2, 2015
Merged

Return array of DataObjects if such is requested.#546
ycombinator merged 9 commits intorackspace:workingfrom
ycombinator:gh-545

Conversation

@ycombinator
Copy link
Copy Markdown
Contributor

In #545, I provided a workaround for what the user was requesting. This PR provides a more robust solution.

Fixes #545.

Attn: @jamiehannaford

@jamiehannaford
Copy link
Copy Markdown
Contributor

LGTM - not sure why Travis is taking so long...

@ycombinator
Copy link
Copy Markdown
Contributor Author

@jamiehannaford I've made all the changes per the code review.

@jamiehannaford
Copy link
Copy Markdown
Contributor

@ycombinator feel free to self-merge this when travis passes 👍

@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage increased (+0.03%) to 88.25% when pulling 22aabaf on ycombinator:gh-545 into e8c07d1 on rackspace:working.

@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage increased (+0.03%) to 88.25% when pulling 3ae1c70 on ycombinator:gh-545 into e8c07d1 on rackspace:working.

@ycombinator
Copy link
Copy Markdown
Contributor Author

Self-merging as Travis build has passed.

ycombinator added a commit that referenced this pull request Mar 2, 2015
Return array of DataObjects if such is requested.
@ycombinator ycombinator merged commit eaeb0fc into rackspace:working Mar 2, 2015
@ycombinator ycombinator deleted the gh-545 branch March 2, 2015 17:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Possible to get DataObject from Guzzle response?

3 participants