Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support dereferencing #41

Merged
merged 5 commits into from
Aug 3, 2017
Merged

Support dereferencing #41

merged 5 commits into from
Aug 3, 2017

Conversation

georgiana-b
Copy link
Contributor

@georgiana-b georgiana-b commented Jul 31, 2017

Fixes #38

Changes made in this PR:

  • remove support for JSON references in Schema

  • support for dereferencing an object's path and URL values

  • dereference Resource descriptor's schema and dialect

  • refactor Package class

  • remove unnecessary distinction between local, remote and inline Resources

I initially also developed support for JSON references, but that turned out to be useless for now. However, I pushed that code to feature/dereferencing-JSON-pointers in case it will be needed in the future.

@georgiana-b georgiana-b force-pushed the feature/support-dereferencing branch 3 times, most recently from 350f48c to 56e5d52 Compare August 1, 2017 13:18
@georgiana-b georgiana-b requested a review from roll August 2, 2017 09:16
@roll
Copy link
Member

roll commented Aug 2, 2017

@georgiana-b
Thanks. I'll review in a few hours.

@georgiana-b georgiana-b mentioned this pull request Aug 3, 2017
Copy link
Member

@roll roll left a comment

Choose a reason for hiding this comment

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

Looks good!

@@ -133,15 +133,15 @@ By default, the gem uses the standard [Data Package Schema](http://specs.frictio
The gem comes with schemas for the standard Data Package Schema, as well as the [Tabular Data Package Schema](http://specs.frictionlessdata.io/tabular-data-package/), and the [Fiscal Data Package Schema](http://fiscal.dataprotocols.org/spec/). These can be referred to via an identifier, expressed as a symbol.

```ruby
package = DataPackage::Package.new(nil, :tabular) # Or :fiscal
package = DataPackage::Package.new(nil, schema: 'tabular-data-package') # Or 'fiscal-data-package'
Copy link
Member

Choose a reason for hiding this comment

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

Removing schema argument will be in next chunk of changes? Because now it's a part of descriptor - descriptor.profile.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yup, that part is in #43.

@georgiana-b georgiana-b merged commit 5a84097 into master Aug 3, 2017
@georgiana-b georgiana-b removed the review label Aug 3, 2017
@georgiana-b georgiana-b deleted the feature/support-dereferencing branch August 3, 2017 11: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.

2 participants