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

Improve typings by allowing JSONSchema6 objects #71

Merged
merged 1 commit into from
Feb 26, 2018

Conversation

krenor
Copy link
Contributor

@krenor krenor commented Feb 5, 2018

I've found no restriction which only applies to JSONSchema4 objects.

Also @types/node have been added to dev-dependencies since Buffer can't be found without them.

@coveralls
Copy link

coveralls commented Feb 5, 2018

Coverage Status

Coverage remained the same at 95.238% when pulling 5a6fbedc3278ce7e71357cd7891460dc17b04c74 on krenor:improve-typings into cc40cb5 on BigstickCarpet:master.

@coveralls
Copy link

coveralls commented Feb 5, 2018

Coverage Status

Coverage remained the same at 95.238% when pulling 3693ea4 on krenor:improve-typings into cc40cb5 on BigstickCarpet:master.

@krenor
Copy link
Contributor Author

krenor commented Feb 6, 2018

@BigstickCarpet This seems like an Issue with Travis setup regarding yarn.. should I remove the yarn.lock file?

@JamesMessinger
Copy link
Member

Hi @krenor. Thanks for the PR. Sorry for the delayed response.

Yes, please remove the yarn.lock file. I think that should make the Travis CI build pass.

Lemme know once that's done and I'll happily merge this PR. Thanks for your work!!! 👍

@JamesMessinger JamesMessinger merged commit 303c892 into APIDevTools:master Feb 26, 2018
@JamesMessinger
Copy link
Member

This PR has been released in v4.1.1 🙌

@krenor krenor deleted the improve-typings branch March 5, 2018 09:41
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.

None yet

3 participants