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

Update to pydantic v1 #28

Merged
merged 6 commits into from
Jan 9, 2020
Merged

Conversation

CasperWA
Copy link
Collaborator

@CasperWA CasperWA commented Jan 8, 2020

Fixes #27

This will update this package to 0.3.1.

Fixes relevant for pydantic v1.

Minor fixes include:

  • Adapting pytest.ini to not show irrelevant warnings.
  • Implementing the same as was done for Update OpenAPI tags Materials-Consortia/optimade-python-tools#121, i.e., fix OpenAPI headlines by fixing endpoint tags in the routers.
  • Update the mappers. Specifically introduce TOP_LEVEL_NON_ATTRIBUTES_FIELDS and remove irrelevant code from the structure mapper.
  • Remove a few pylint disable comments.

Use TOP_LEVEL_NON_ATTRIBUTES_FIELDS to define all non-`attributes`
fields in an entry.

Remove irrelevant code from structures mapper.
pydantic's `AnyUrl` does not support a `?` at the end of a URL.
@CasperWA CasperWA merged commit ec0889b into aiidateam:develop Jan 9, 2020
@CasperWA CasperWA deleted the update_pydantic_v1 branch January 9, 2020 10:56
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.

Update to pydantic v1
1 participant