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

Fix issue with ClassVar parsing #3403

Merged
merged 4 commits into from Nov 18, 2021
Merged

Conversation

uriyyo
Copy link
Contributor

@uriyyo uriyyo commented Nov 10, 2021

Change Summary

Related issue number

Fixes #3401

Checklist

  • Unit tests for the changes exist
  • Tests pass on CI and coverage remains at 100%
  • Documentation reflects the changes where applicable
  • changes/<pull request or issue id>-<github username>.md file added describing change
    (see changes/README.md for details)
  • My PR is ready to review, please add a comment including the phrase "please review" to assign reviewers

@uriyyo
Copy link
Contributor Author

uriyyo commented Nov 10, 2021

please review

@uriyyo uriyyo force-pushed the fix-issue-3401 branch 3 times, most recently from 1efd532 to 0535148 Compare November 10, 2021 14:16
@PrettyWood
Copy link
Member

Thank you so much

@PrettyWood PrettyWood merged commit ee0b2e2 into pydantic:master Nov 18, 2021
@dusty-phillips
Copy link

Is there a release planned to get this on pypi?

jpribyl pushed a commit to liquet-ai/pydantic that referenced this pull request Dec 4, 2021
* Fix issue with ClassVar parsing

* Fix lint error

* Simplify test

* Fix condition and test
PrettyWood pushed a commit to PrettyWood/pydantic that referenced this pull request Dec 6, 2021
* Fix issue with ClassVar parsing

* Fix lint error

* Simplify test

* Fix condition and test
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Pydantic fail to parse ClassVar annotation when it's a string
5 participants