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

Data Package spec says resources is not required #434

Closed
pwalsh opened this Issue May 26, 2017 · 9 comments

Comments

Projects
None yet
3 participants
@pwalsh
Member

pwalsh commented May 26, 2017

However, at some point we agreed it should be.

@pwalsh pwalsh added the Data Package label May 26, 2017

@pwalsh pwalsh added this to the v1.0 milestone May 26, 2017

@pwalsh pwalsh referenced this issue Jun 15, 2017

Closed

Profiles "spec" #435

@pwalsh

This comment has been minimized.

Show comment
Hide comment
@pwalsh

pwalsh Jun 15, 2017

Member

Depends on #435

Member

pwalsh commented Jun 15, 2017

Depends on #435

@pwalsh

This comment has been minimized.

Show comment
Hide comment
@pwalsh
Member

pwalsh commented Jun 19, 2017

@roll

This comment has been minimized.

Show comment
Hide comment
@roll

roll Jun 19, 2017

Member

http://specs.frictionlessdata.io/data-package/

The descriptor MUST contain a resources property describing the data resources.

It seems it's missed only in Metadata. Required properties but current wording kinda separate it (metadata and resources).

Member

roll commented Jun 19, 2017

http://specs.frictionlessdata.io/data-package/

The descriptor MUST contain a resources property describing the data resources.

It seems it's missed only in Metadata. Required properties but current wording kinda separate it (metadata and resources).

@rufuspollock

This comment has been minimized.

Show comment
Hide comment
@rufuspollock

rufuspollock Jun 21, 2017

Contributor

@pwalsh where is it missing? I agree it is a MUST.

Contributor

rufuspollock commented Jun 21, 2017

@pwalsh where is it missing? I agree it is a MUST.

@pwalsh pwalsh self-assigned this Jun 21, 2017

@pwalsh

This comment has been minimized.

Show comment
Hide comment
@pwalsh

pwalsh Jun 21, 2017

Member

I'll fix.

Member

pwalsh commented Jun 21, 2017

I'll fix.

@rufuspollock

This comment has been minimized.

Show comment
Hide comment
@rufuspollock

rufuspollock Jun 23, 2017

Contributor

@pwalsh i was asking where it was missing 😉 - not suggesting we needed a change. I've commented on the PR #467.

Contributor

rufuspollock commented Jun 23, 2017

@pwalsh i was asking where it was missing 😉 - not suggesting we needed a change. I've commented on the PR #467.

@rufuspollock rufuspollock reopened this Jun 23, 2017

@pwalsh

This comment has been minimized.

Show comment
Hide comment
@pwalsh

pwalsh Jul 3, 2017

Member

@rufuspollock I've read everything in #467 and the distinctions you are making about different types of metadata are very specific and don't make a lot of sense to me. Can you please do the PR to get this how you want it to be.

Member

pwalsh commented Jul 3, 2017

@rufuspollock I've read everything in #467 and the distinctions you are making about different types of metadata are very specific and don't make a lot of sense to me. Can you please do the PR to get this how you want it to be.

@pwalsh pwalsh assigned rufuspollock and unassigned rufuspollock and pwalsh Jul 3, 2017

@rufuspollock

This comment has been minimized.

Show comment
Hide comment
@rufuspollock

rufuspollock Jul 9, 2017

Contributor

@pwalsh let's chat quickly about this please - i think 5m on a call will save 30m in issues 😉

Contributor

rufuspollock commented Jul 9, 2017

@pwalsh let's chat quickly about this please - i think 5m on a call will save 30m in issues 😉

@rufuspollock

This comment has been minimized.

Show comment
Hide comment
@rufuspollock

rufuspollock Jul 30, 2017

Contributor

As we did not get the 5m chat I'm leaving this closed for now /cc @pwalsh

Contributor

rufuspollock commented Jul 30, 2017

As we did not get the 5m chat I'm leaving this closed for now /cc @pwalsh

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