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

Allow digits in property names in hyper schema #316

Merged

Conversation

tsrivishnu
Copy link
Contributor

Similar to #284

Couldn't see a reason why the digits are not allowed.

@tsrivishnu
Copy link
Contributor Author

Didn't see a way to override the validation or the schema, otherwise, we wouldn't have to change the gem.

@geemus geemus merged commit 3698bd2 into interagent:master Dec 6, 2016
@geemus
Copy link
Member

geemus commented Dec 6, 2016

Thanks!

@steinnes
Copy link

steinnes commented Dec 6, 2016

Wow, I just came across this error:

#/definitions/address/properties: failed schema #/definitions/resource/properties/properties: "address1", "address2" are not permitted keys.

My team mates were very surprised when I showed them the PR to fix this was merged 3 hours ago 😸 ... back to using prmd installed from git =)

@michaelsauter
Copy link
Contributor

... and on that note - could we get a new release? I'd be keen on this MR but also the recent TOC stuff.

Thanks!

@geemus
Copy link
Member

geemus commented Dec 8, 2016

released as v0.13.0

@michaelsauter
Copy link
Contributor

Thanks heaps!

@tsrivishnu tsrivishnu deleted the allow-digits-in-property-names branch December 20, 2016 10:58
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