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 json parse error on Godeps #746

Merged
merged 1 commit into from
Jul 24, 2015
Merged

fix json parse error on Godeps #746

merged 1 commit into from
Jul 24, 2015

Conversation

squarism
Copy link
Contributor

Deleted brace was causing an error on master. Fixed tabs/spaces at the same time.

Test:

jq . Godeps/Godeps.json
parse error: ':' not as part of an object at line 85, column 16

Signed-off-by: Chris Dillon <squarism@gmail.com>
@RichardScothern
Copy link
Contributor

Thanks @squarism . Please sign your commits and I will merge.

@RichardScothern RichardScothern self-assigned this Jul 24, 2015
@RichardScothern RichardScothern added this to the Registry/2.1 milestone Jul 24, 2015
@squarism
Copy link
Contributor Author

Apologies for not reading the contributing.md well. 🍰

RichardScothern pushed a commit that referenced this pull request Jul 24, 2015
fix json parse error on Godeps
@RichardScothern RichardScothern merged commit 94548b8 into distribution:master Jul 24, 2015
@stevvooe
Copy link
Collaborator

@squarism Thank you for the fix.

Looks like this slipped through with the swift driver merge. @lebauce

@squarism
Copy link
Contributor Author

Np! I'm just excited about the recent #461 being implemented. Great job! 🎉

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