Added validation_error_cause to config.#7626
Added validation_error_cause to config.#7626sydney-runkle merged 4 commits intopydantic:mainfrom zakstucke:main
Conversation
|
please review |
|
Thanks @zakstucke for this patch 🙏 It would be great to have at least one test for this config flag. Please update |
|
@hramezani its got quite a few different pathways tested fully in pydantic core. Do you want me to add exception group as a test dep in pydantic as well and test all the different pathways, or something simpler? Pathways:
|
|
@zakstucke I am thinking about having only the last option( |
|
@hramezani ok sounds good, just added test for 3.11 modern pathway, disabled and enabled. Please review |
|
Thanks @zakstucke for refreshing the PR, we will include this in the next release |
sydney-runkle
left a comment
There was a problem hiding this comment.
Looks great! Thanks @zakstucke for the contribution 😄
|
@zakstucke, do you want to change the description of this PR to have Fix #6498? instead of just the issue reference? |
|
@sydney-runkle done! (If I understood you correctly) |
Change Summary
Hooks up
validation_error_cause: boolconfig value added to pydantic core.Related issue number
Fix #6498
Checklist
Selected Reviewer: @hramezani