Comments (4)
Also, editor.swagger.io's petstore-simple has no type: object
for all its definitions.
from bravado-core.
As per ron from swagger: it's not mandatory in schema definition, but it is highly recommended (I'll fix the examples). Not having a type has an unexpected meaning to most users.
from bravado-core.
The fix for #35 makes it so 'type' is required. The rationale being, definitions split up across multiple files are a little more involved to go track down and check for compliance. This is no longer just a matter of checking #/definitions
in the root spec.
from bravado-core.
Why can't we just guess the type based on the given keywords if it is not defined?
http://json-schema.org/latest/json-schema-validation.html
e.g.
properties -> object
items -> array
else -> string
I'm using Restlet Studio for designing my API and it also doesn't generate these 'type' values.
(I already dropped them a mail, but I think there might be more tools which could make problems here)
from bravado-core.
Related Issues (20)
- Spec.from_dict() fails to return models in spec.definitions HOT 4
- Spec.from_dict() fails with a local file uri as origin_url HOT 5
- Mapping operationId with paths and operations
- Travis build is failing on python2.7 after jsonschema 3.0.0 release HOT 2
- bravado-core assumes that response body is present if a body parameter is defined
- bravado-core does propagate x-nullable field in case of reference objects
- Missing API reference? HOT 1
- Validation fails for x-nullable objects that happen to have discriminator HOT 2
- Add pickle support to bravado_core.spec.Spec
- uuid format is not registered with bravado-core HOT 2
- collectionFormat array list of lists
- jsonschema processing doesn't support jsonschema>=4.0.0 causing bravado to break when working in offline environment HOT 2
- jsonschema deprecation warning
- 5.17.1: pytest runs forever in `tests/spec/Spec/deref_flattened_spec_test.py::test_build_spec_object_with_recursive_specs_and_fully_dereference` unit HOT 7
- Tests hang HOT 3
- Dereference ref_dict causes hang HOT 1
- RFE: please start making github releases
- GPL Violation? HOT 1
- Warning: Wrong tag v6.6.1
- RFE: use `zoneinfo` instead of `pytz`
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from bravado-core.