Skip to content
This repository was archived by the owner on Jun 18, 2024. It is now read-only.

Format became an array at top level and a string inside distribution #145

Closed
JoshData opened this issue Sep 20, 2013 · 3 comments
Closed

Comments

@JoshData
Copy link
Contributor

Not sure what to make of this change. The "format" field is now supposed to be an array of MIME types. If the top-level format field is supposed to correspond to the top-level accessURL, then it would only ever be a single MIME type since accessURL necessarily provides a single downloadable file. (Except in unusual and advanced use of HTTP.)

Or if it repeats the formats found in the distributions, then it's redundant.

It's also at odds with the example in the documentation for "distribution" which shows the use of distribution's "format" field as a string, and not an array.

So... just trying to figure out what I'm supposed to put in it....

Congrats on posting the updated spec though!

@haleyvandyck
Copy link
Contributor

Interesting-- thanks for pointing this out. Thoughts @MarinaMartin, @seanherron?

Thanks for all your help with the schema updates along the way, @JoshData.

@seanherron
Copy link
Contributor

This is a good point. I believe it may have been made when accessURL was proposed to be an array of URLs. I'll submit a pull request to change the cardinality and format.

@gbinal
Copy link
Contributor

gbinal commented Nov 4, 2013

It looks like #148 fixed, so I'm closing this issue. Josh, thanks for the heads up.

@gbinal gbinal closed this as completed Nov 4, 2013
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants