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

Add Simple CI test suite #107

Open
danfinlay opened this issue Jun 27, 2016 · 2 comments
Open

Add Simple CI test suite #107

danfinlay opened this issue Jun 27, 2016 · 2 comments

Comments

@danfinlay
Copy link
Collaborator

Should be an easy enough contribution, maybe I'll get to it soon.

I accidentally merged someone's JSON with a trailing comma, if we just validated the JSON automatically that wouldn't be a problem!

We could even check that there's a file for each vector & raster listed.

@wamserma
Copy link

I made a start in PR #169.

Checking for existence of files is another step

@scottmmjackson
Copy link
Contributor

#249 is a metadata file verification script. Currently it's warn-only because of the number of extant defects:

  • name is frequently an improper slug
  • vector is frequently missing or bogus
  • filename isn't a known field but is present anyway

The next step will be sweeping through the defects and fixing them, then making defects fail CI so people can make proper submissions mostly-unsupervised.

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

No branches or pull requests

3 participants