Skip to content

Rework name resolver#476

Merged
lafrech merged 3 commits intodevfrom
rework-name-resolver
Aug 21, 2019
Merged

Rework name resolver#476
lafrech merged 3 commits intodevfrom
rework-name-resolver

Conversation

@Bangertm
Copy link
Copy Markdown
Collaborator

Changes the schema_name_resolver function to be able to accept either a Schema class, Schema instance or a string that resolves to a Schema class.

resolves #475

@lafrech lafrech self-requested a review July 19, 2019 23:17
Copy link
Copy Markdown
Member

@lafrech lafrech left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Thanks.

@lafrech
Copy link
Copy Markdown
Member

lafrech commented Jul 19, 2019

I think this is gonna be a 3.0.0 since it changes what the resolver can receive.

Bangertm added 3 commits July 31, 2019 10:51
Custom resolution of a schema class can now be done directly in a
custom schema_name_resolver function
@Bangertm Bangertm force-pushed the rework-name-resolver branch from a594ef7 to d880079 Compare July 31, 2019 14:52
@lafrech
Copy link
Copy Markdown
Member

lafrech commented Aug 21, 2019

Sorry @Bangertm for the delay. I just reviewed this again and I think it is good to go.

Thanks.

@lafrech lafrech merged commit 78b1480 into dev Aug 21, 2019
@lafrech
Copy link
Copy Markdown
Member

lafrech commented Aug 21, 2019

This will be a 3.0.0.

If we don't manage to review #478 quickly to cram it into the release, I'll release 3.0.0 without it and it will be 3.1.0 (or 4.0.0 if it is not backwards compatible).

@sloria sloria deleted the rework-name-resolver branch January 19, 2024 04:28
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.

partial=True on schema issues non-unique warning

2 participants