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 argument name collision #1367
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1367 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 21 21
Lines 3723 3723
Branches 735 735
=========================================
Hits 3723 3723
Continue to review full report at Codecov.
|
LGTM, I'll do my best to get this merged when I next have time to go through PRs. |
thanks so much, sorry again for the delay. |
Fixes awslabs#767 The `model_name` argument was renamed to `__model_name` in [pydantic/pydantic#1367. This change ensure that our code works with both future and past versions of pydantic.
Fixes awslabs#767 The `model_name` argument was renamed to `__model_name` in [1367](pydantic/pydantic#1367). This change ensure that our code works with both future and past versions of pydantic.
Fixes #767 The `model_name` argument was renamed to `__model_name` in [1367](pydantic/pydantic#1367). This change ensure that our code works with both future and past versions of pydantic.
Fixes awslabs#767 The `model_name` argument was renamed to `__model_name` in [1367](pydantic/pydantic#1367). This change ensure that our code works with both future and past versions of pydantic.
* Fix pydanitc `create_model` usage. (#768) Fixes #767 The `model_name` argument was renamed to `__model_name` in [1367](pydantic/pydantic#1367). This change ensure that our code works with both future and past versions of pydantic. * fix style according to black==19.3b0 Co-authored-by: Jasper Schulz <schjaspe@amazon.de>
Fixes #767 The `model_name` argument was renamed to `__model_name` in [1367](pydantic/pydantic#1367). This change ensure that our code works with both future and past versions of pydantic.
Change Summary
Renaming
model_name
argument ofmain.create_model()
to__model_name
to allow usingmodel_name
as a field name.Related issue number
#1366
Checklist
changes/<pull request or issue id>-<github username>.md
file added describing change(see changes/README.md for details)