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.

Sign up for GitHub

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

Jump to bottom

Local Reference Fixes 0.9.0 #15

Draft
wants to merge 32 commits into
base: main
Choose a base branch
from
Draft

Local Reference Fixes 0.9.0 #15

wants to merge 32 commits into from

Conversation

p1-ra
Copy link

@p1-ra p1-ra commented May 11, 2021

@p1-ra p1-ra self-assigned this May 11, 2021
@p1-ra p1-ra force-pushed the main-with-fixes branch from c870943 to a751609 Compare May 12, 2021 09:42
forest-benchling and others added 4 commits May 12, 2021 08:38
@forest-benchling
fix: Deserialization of optional nullable properties when no value is…
e708750 
… returned from the API [openapi-generators#420 & openapi-generators#381]. Thanks @forest-benchling!

* Handle optional nullable deserialization

* Fix test
@dbanty
fix(parser): Prevent crash when providing a non-string default to a s…
fddf68a 
…tring attribute. Fixes openapi-generators#414 (openapi-generators#415)
@dbanty @Nementon
feat(parser): Allow references to non-object, non-enum types [openapi…
79acbf8 
…-generators#371][openapi-generators#418][openapi-generators#425]. Thanks @p1-ra!

* parser / properties / do not restrict reference pointer type to `enum` or `object`

* e2e / update `openapi.json` and golden-record

* tpl / model / add missing typing import: `Tuple,Optional,BinaryIO,TextIO`

* test: Make big ref test model properties required to lessen generated code

Co-authored-by: Nementon <[email protected]>
@dbanty
chore: Prep 0.9.1 release
0f58cfd
@p1-ra p1-ra force-pushed the main-with-fixes branch 2 times, most recently from 44183cd to d9c73e2 Compare May 14, 2021 12:10
p1-ra and others added 20 commits May 17, 2021 17:04
@p1-ra
templates / endpoint_module / form_data: do not use attr.asdict, us…
588fa23 
…e <Model>.to_dict()` serializer

   - to handle internal UNSET values to not serialize them,
     `attr.asdict` do not know UNSET type and will serialize them
@p1-ra
e2e / update golden-record
1b17d1f
@Nementon @p1-ra
templatize api,endpoint init files
907208e
@p1-ra
e2e / add {api_init.py.jina, endpoint_init.py.jinja} custom templates…
15c7b13 
… tests
@Nementon @p1-ra
expose utils module to jinja2
df9ac8a
@Nementon @p1-ra
test_custom_templates / replace jinja2 marcro with call to utils module
8135d38
@p1-ra
task / regen / regen custom template golden record
0b43462
@Nementon @p1-ra
updated vendored openapi_schema_pydantic / correct oai false negative
074b58d 
validation
@Nementon @p1-ra
vendored openapi_schema_pydantic / correct flake8 warnings
1e0620c 
    - correct: W293, W291, F821
@Nementon @p1-ra
correct openapi_schema_pydantic schemas breaking changes
d8c1a9f 
  - anyOf, allOf, oneOf now default to `None` instead
    of `[]`
@p1-ra
update unit tests
5befdc5
@p1-ra
parser / properties / avoid call to isinstance for perf imprv
470be5f
@p1-ra
openapi_schema_pydantic / schema / set nullable type to bool
b09ebd6
@p1-ra
openapi_schema_pydantic / remove ellipsis notation
e71c4cc 
  - Mypy do not well support it to declare `undefined` value
@p1-ra
openapi_schema_pydantic / allow extra values
2902c21 
  - For support of OAI vendor extension
@p1-ra
openapi_schema_pydantic / correct typing
59b6e5e
@p1-ra
openapi_schema_pydantic / schema / {any,all,one}Of default to `List.E…
9371b1b 
…mpty`
@p1-ra
parser / schemas / add indirect reference resolution
0d66065
@p1-ra
e2e / update openapi.json and golden-record
58fa3de
@p1-ra
parser / property / add recursive reference resoltion
4d67fe5
@p1-ra p1-ra force-pushed the main-with-fixes branch from d9c73e2 to 6d4416d Compare May 18, 2021 10:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@p1-ra p1-ra

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@p1-ra @christhekeele @forest-benchling @dbanty @Nementon