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 inheritance logic when intermediate class is present #5

Merged

Conversation

thomasmarwitz
Copy link
Contributor

@thomasmarwitz thomasmarwitz commented Nov 5, 2024

This fixes #4

Copy link
Member

@pawamoy pawamoy left a comment

Choose a reason for hiding this comment

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

Could you add a new test rather than modify an existing one?

@thomasmarwitz thomasmarwitz force-pushed the fix-intermediate-inheritance branch 2 times, most recently from a8e0884 to 5762c89 Compare November 5, 2024 13:20
@pawamoy
Copy link
Member

pawamoy commented Nov 5, 2024

I tried this locally, and unfortunately the test passes even without the fix.

@thomasmarwitz thomasmarwitz force-pushed the fix-intermediate-inheritance branch from 5762c89 to f5eacd5 Compare November 5, 2024 13:22
@thomasmarwitz thomasmarwitz requested a review from pawamoy November 5, 2024 13:24
tests/test_extension.py Outdated Show resolved Hide resolved
pawamoy and others added 5 commits November 5, 2024 14:27
…tz/griffe-inherited-docstrings into fix-intermediate-inheritance
Co-authored-by: Timothée Mazzucotelli <dev@pawamoy.fr>
…tz/griffe-inherited-docstrings into fix-intermediate-inheritance
Copy link
Member

@pawamoy pawamoy left a comment

Choose a reason for hiding this comment

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

OK LGTM, thank you for following up 😄

@thomasmarwitz
Copy link
Contributor Author

Thanks for double-checking everything :)

@pawamoy pawamoy merged commit 639ff80 into mkdocstrings:main Nov 5, 2024
25 checks passed
@pawamoy
Copy link
Member

pawamoy commented Nov 5, 2024

Released in v1.1.1.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

bug: Inheritance breaks when intermediate class doesn't declare member
2 participants