Skip to content

Make shadowing attributes a warning instead of an error#7193

Merged
samuelcolvin merged 2 commits intomainfrom
make-shadowing-attribute-a-warning
Aug 22, 2023
Merged

Make shadowing attributes a warning instead of an error#7193
samuelcolvin merged 2 commits intomainfrom
make-shadowing-attribute-a-warning

Conversation

@adriangb
Copy link
Copy Markdown
Member

@adriangb adriangb commented Aug 21, 2023

Closes #7108

Selected Reviewer: @samuelcolvin

@adriangb
Copy link
Copy Markdown
Member Author

please review

@cloudflare-workers-and-pages
Copy link
Copy Markdown

cloudflare-workers-and-pages Bot commented Aug 21, 2023

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: 7584523
Status: ✅  Deploy successful!
Preview URL: https://68724c3c.pydantic-docs2.pages.dev
Branch Preview URL: https://make-shadowing-attribute-a-w.pydantic-docs2.pages.dev

View logs

Comment thread tests/test_main.py Outdated
Copy link
Copy Markdown
Member

@samuelcolvin samuelcolvin left a comment

Choose a reason for hiding this comment

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

LGTM

@samuelcolvin samuelcolvin merged commit caa7801 into main Aug 22, 2023
@samuelcolvin samuelcolvin deleted the make-shadowing-attribute-a-warning branch August 22, 2023 18:03
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.

shadows an attribute in parent __pydantic_init_subclass__ setattr

3 participants