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

Added more verbose JSON schema validation messages #70

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

ProLoser
Copy link
Collaborator

@ProLoser ProLoser commented Feb 3, 2016

Those guys at @CINEPASS did an awesome job adding what I've found to be a super critical tweak.

@ProLoser
Copy link
Collaborator Author

ProLoser commented Feb 5, 2016

@cybertk If you make me a collaborator I will try to resolve the failing tests and move the repo along

@plroebuck
Copy link
Collaborator

Looks like the tests were passing, but there were load issues with the site itself. Also seemed to be having issues installing the CINEPASS tv fork. Can we retry the CI and see new results?

@cybertk
Copy link
Owner

cybertk commented Feb 8, 2016

Add both of you as collaborator, @ProLoser @plroebuck. Thanks for your contributions

@cybertk
Copy link
Owner

cybertk commented Aug 3, 2016

@ProLoser Could you please help complete this PR?

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

Successfully merging this pull request may close these issues.

4 participants