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

Profile field includes version as # and profile is url (?) #491

Closed
rufuspollock opened this issue Jul 14, 2017 · 4 comments
Closed

Profile field includes version as # and profile is url (?) #491

rufuspollock opened this issue Jul 14, 2017 · 4 comments

Comments

@rufuspollock
Copy link
Contributor

Extend profile field to include version of the profile using # e.g.

profile:data-package#v1.0

Also consider explicit urls for profile schema:

profile: url-to-schema

cf ropensci-archive/datapkg#12 (comment)

@ezwelty
Copy link
Contributor

ezwelty commented Jul 14, 2017

I really think this should be a url-or-path, and nothing more. A url allows custom profiles and skips the fragile step of parsing "data-package#v1.0" to a url, and path allows the profile to be included within the data package.

I liked the suggestion of this being called schema, since that's what's being pointed to.
edit: Oops, sorry, schema is already a data resource property, so having it mean two different things would be confusing.

@rufuspollock
Copy link
Contributor Author

@ezwelty really like the suggestion and you've grokked why we don't use schema (which would anyway be ambiguous over data schema vs metadata schema).

@pwalsh
Copy link
Member

pwalsh commented Jul 24, 2017

@rufuspollock I much prefer the solution I have for versions, scheduled for discussion for v1.1, here:

As for URL to schema, I was supporting this already in rc1:

@pwalsh pwalsh added this to the v1.1 milestone Jul 24, 2017
@rufuspollock
Copy link
Contributor Author

@pwalsh another reason why you and I having regular chats is a "good idea" (tm) 😉

Again the question of "normalized" form i.e. two properties vs all in one string is there.

Anyway I'm going to close this as DUPLICATE in favour of #444.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

No branches or pull requests

3 participants