Skip to content
This repository has been archived by the owner on Feb 11, 2021. It is now read-only.

Validate json #21

Closed
jimaek opened this issue Jul 5, 2014 · 5 comments
Closed

Validate json #21

jimaek opened this issue Jul 5, 2014 · 5 comments

Comments

@jimaek
Copy link
Member

jimaek commented Jul 5, 2014

It would help a lot if the bot could validate update.json and comment any errors it finds.

@megawac
Copy link
Contributor

megawac commented Jul 22, 2014

jsdelivr/jsdelivr#1273

@megawac
Copy link
Contributor

megawac commented Jul 23, 2014

How comprehensive do you think this check should be @jimaek. If I spend time implementing this I'm probably going to make it report all the files that would be included from the last release (if the repo is a github one)

@jimaek
Copy link
Member Author

jimaek commented Jul 23, 2014

The easiest check would be to simply check if the json is valid.
What other stuff did you want to include?

@megawac
Copy link
Contributor

megawac commented Jul 23, 2014

I'd probably do this one in conjunction with #26 and provide collabs with all the files that would be included if the next release contains the same files as the previous

@jimaek
Copy link
Member Author

jimaek commented Jul 23, 2014

Currently my issue with update.json is the manual json verification that I need to do, If you can add this I will be a happy man :)
Except from that you can add anything else you want. This project is your baby :)

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

No branches or pull requests

2 participants