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

Json reference #21

Merged
merged 23 commits into from
Dec 14, 2014
Merged

Json reference #21

merged 23 commits into from
Dec 14, 2014

Conversation

mission-liao
Copy link
Member

No description provided.

- when upgrade from 1.2, now skip basePath part, everything after host
would be used as path.
remove SwaggerApp.__root_url
replace SwaggerApp.__url2obj with SwaggerApp.__url2app
remove SwaggerApp._prepare_url, instead, all input url would be refined
by utils.normalize_url
decompose json-reference into (normalized-url, json-pointer)
- add SwaggerApp.__app_cache to union multiple SwaggerApp with url
- add SwaggerApp.__url_load_hook to patch url to load json.
- 'getter' parameter in SwaggerApp.create is removed, if needy, call
SwaggerApp.load instead.
- all $ref are normalized to full JSON referenceall $ref are normalized
to full JSON reference
- separate scanners: Resolve, PatchObject into 2 runs to make sure the
order is expected.
- return ('', '#') when input == '#' for jr_split
- refine get_swagger_version for partial swagger json
to recognize version of swagger for partial swagger.json
swagger.json with PathItem and Schema is allowed.
@coveralls
Copy link

Coverage Status

Coverage increased (+0.08%) when pulling 802fd21 on json_reference into 208fc87 on master.

mission-liao added a commit that referenced this pull request Dec 14, 2014
@mission-liao mission-liao merged commit bc4ffcd into master Dec 14, 2014
@mission-liao mission-liao deleted the json_reference branch December 14, 2014 09:55
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.

2 participants