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

Use an existing CSV schema format? #13

Closed
harrisj opened this issue Mar 12, 2016 · 3 comments
Closed

Use an existing CSV schema format? #13

harrisj opened this issue Mar 12, 2016 · 3 comments

Comments

@harrisj
Copy link

harrisj commented Mar 12, 2016

This is pretty awesome, and what I'm suggesting is possibly overkill, but I was wondering if you had considered using one of the CSV schema formats for specifying the fields in the CSV. These seem to be the two biggest ones out there:

I will admit this seems like a bit of overkill for a CSV of states, but it might be useful if you wanted to automatically validate future changes or additions with an automated test and then you get CI for your CSV. For instance, Goodtables is a validator that uses the JSON schema format (although it needs some work). CSVLint is another new entrant I haven't evaluated it but it also uses the JSON schema format (which seems like the one to consider now).

@onyxfish
Copy link
Collaborator

I did briefly consider this. Too be perfectly honest, I mostly just did what was expedient. Some things, like the history of contributors, don't obviously fit cleanly into those other formats, but they could probably be made to work. In general, I don't see much advantage in standards for these kinds of one-off microformats, however, your example of automated validation is a compelling case. I'll def consider it for the first breaking release, whenever that ends up being.

@harrisj
Copy link
Author

harrisj commented Mar 13, 2016

I don't have a strong opinion on it myself and I'd rather have something real rather than waiting on conformance so it's up to you. I guess I'd also want to check if the tools are all they claim too and can be run locally.

@jpmckinney
Copy link
Member

Closing as no bandwidth.

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

No branches or pull requests

3 participants