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

Fix for #36 #37

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Fix for #36 #37

wants to merge 1 commit into from

Conversation

a-h
Copy link

@a-h a-h commented Jul 16, 2016

The error message was actually a gofmt error, the underlying problem was that a missing JSON schema title resulted in invalid go code.

I improved the error descriptions in a few places and defaulted the package name to "main" if the JSON schema title was omitted.

…ng problem was that a missing JSON schema title resulted in invalid go code.
@a-h
Copy link
Author

a-h commented Jul 17, 2016

The Travis CI build is failing because the build attempts to pull down https://code.google.com/p/go.tools/cmd/cover - but that's now a redirect to https://github.com/golang/go and the build configuration needs to be updated. Go development switched to github in 2014.

@josegonzalez
Copy link

Any chance this can be merged? This fixes an issue I'm having locally with this library.

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.

2 participants