-
Notifications
You must be signed in to change notification settings - Fork 65
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
Support for using a JSON string rather than a URL path? #58
Comments
Hi @jszwedko, thanks for getting in touch! I've had a look at your use case from the sfhomeless-api. We have run into problems with types and The general feature would be useful, and definitely accepted. The main thing we ask is that there are adequate tests provided for any new feature (see our contributing guidelines ) I might need to look into how others are using I'm happy to discuss further and review code etc -- I'm glad you are finding apivore useful in your project! Cheers! |
Hey! Apologies for the delay. Yeah I was thinking, rather than adding support for I'm also mixed about supporting I'll put together a PR for using a string rather than a route. I definitely appreciate the increased confidence apivore gives me that my API is complying with the spec. |
I have a scenario where I need to preprocess the swagger.json before giving it to apivore to validate (translating
x-nullable
to adding"null"
to the type list for a given property). I can take a crack at implementing this, but wanted to see if it the change would be accepted first.Thank you!
The text was updated successfully, but these errors were encountered: