Skip to content
This repository was archived by the owner on Feb 15, 2018. It is now read-only.

Feature/includes #16

Merged
merged 4 commits into from
Sep 4, 2016
Merged

Feature/includes #16

merged 4 commits into from
Sep 4, 2016

Conversation

madninja
Copy link
Contributor

@madninja madninja commented Sep 4, 2016

  • Renames a few utility functions to reflect what they do better
  • DRY up the factory methods for resource creation
  • Implement include requests and included resource retrieval

@madninja
Adds support for include and use_include in relationships
f40d9b9
@madninja
Some re-factoring
d5a3c00 
* Renames “build_resource” functions to “build_request” since that’s what they do.
* Add _mk_one and _mk_many helper methods to Resource to DRY a bit more
@madninja
Actually validate the use_included request
7ecc400
@coveralls
Copy link

coveralls commented Sep 4, 2016 •
edited
Loading

Coverage Status

Coverage increased (+0.3%) to 96.95% when pulling 7ecc400 on feature/includes into 0603fb4 on master.

@coveralls
Copy link

coveralls commented Sep 4, 2016 •
edited
Loading

Coverage Status

Coverage increased (+0.3%) to 96.95% when pulling 62d8254 on feature/includes into 0603fb4 on master.

@madninja madninja merged commit 06fd048 into master Sep 4, 2016
@madninja madninja deleted the feature/includes branch September 4, 2016 23:48
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@madninja @coveralls