Skip to content

fixed generic inheritance edge case and added test #67

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

Closed

Conversation

bpeake-illuscio
Copy link

@bpeake-illuscio bpeake-illuscio commented Dec 13, 2018

Fixes Type Error that occurs when creating documentation using inherited concrete Generic types as annotations.

Likely closes: https://github.com/agronholm/sphinx-autodoc-typehints/issues/66

Tests have been added for this case. Tests pass on Python 3.6 / OSX

@bpeake-illuscio
Copy link
Author

hmm, just noticed this is a duplicate fix to https://github.com/agronholm/sphinx-autodoc-typehints/pull/60/files.

@agronholm
Copy link
Collaborator

I'll look at this this Sunday.

@bpeake-illuscio
Copy link
Author

bpeake-illuscio commented Dec 23, 2018

I'll look at this this Sunday.

Just curious if you have gotten around to it. I have to disable this plugin for readthedocs right now even though it's super cool since this error kills the build process. Thanks!

@agronholm
Copy link
Collaborator

Sorry, no. I have a couple PRs to review and merge and I'll do them all today.

@agronholm agronholm closed this in 645fa6b Dec 23, 2018
@agronholm
Copy link
Collaborator

I changed the implementation but kept your test. Thank you.

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.

TypeError: 'NoneType' object is not iterable
2 participants