Skip to content
This repository has been archived by the owner on Nov 1, 2023. It is now read-only.

add monkeypatch to hotfix pydantic Union issues #982

Merged
6 commits merged into from
Jun 11, 2021

Conversation

bmc-msft
Copy link
Contributor

Until Pydantic supports discriminated or "smart" unions, we need to work around the coercion issue impacting unions in our models.

This reuses the "smart union" implementation from pydantic/pydantic#2092

@bmc-msft bmc-msft linked an issue Jun 10, 2021 that may be closed by this pull request
src/pytypes/onefuzztypes/_monkeypatch.py Show resolved Hide resolved
src/pytypes/onefuzztypes/_monkeypatch.py Outdated Show resolved Hide resolved
src/pytypes/onefuzztypes/_monkeypatch.py Show resolved Hide resolved
src/pytypes/onefuzztypes/_monkeypatch.py Outdated Show resolved Hide resolved
src/pytypes/onefuzztypes/_monkeypatch.py Show resolved Hide resolved
src/pytypes/onefuzztypes/_monkeypatch.py Outdated Show resolved Hide resolved
@ghost
Copy link

ghost commented Jun 11, 2021

Hello @bmc-msft!

Because this pull request has the auto-merge label, I will be glad to assist with helping to merge this pull request once all check-in policies pass.

p.s. you can customize the way I help with merging this pull request, such as holding this pull request until a specific person approves. Simply @mention me (@msftbot) and give me an instruction to get started! Learn more here.

@ghost ghost merged commit c0b3a40 into microsoft:main Jun 11, 2021
@bmc-msft bmc-msft deleted the add-pydantic-union-hotfix branch June 14, 2021 12:46
@ghost ghost locked as resolved and limited conversation to collaborators Jul 14, 2021
This pull request was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

github issue notification in templates not working
3 participants