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 annotation for directives.other.Include.run #12042

Closed

Conversation

danieleades
Copy link
Contributor

the return type here should be Sequence rather than list since it uses the super class method which is typed as Sequence

@danieleades
Copy link
Contributor Author

the mypy error here is wrong, but it's wrong because the docutils-stubs type is wrong. Meaning it can't be fixed until after migrating to the typeshed stubs.

setting as draft

@danieleades danieleades marked this pull request as draft March 3, 2024 08:20
@picnixz
Copy link
Member

picnixz commented Mar 3, 2024

Could we just ignore this one with an override instead? Or do you think it will be fixed upstream fast enough?

@danieleades
Copy link
Contributor Author

Could we just ignore this one with an override instead? Or do you think it will be fixed upstream fast enough?

hopefully fixed upstream soon enough!

@jayaddison jayaddison added type:enhancement enhance or introduce a new feature docutils Tags upstream Docutils issues internals:typing labels Mar 3, 2024
@AA-Turner
Copy link
Member

Covered in #12012

@AA-Turner AA-Turner closed this Jul 13, 2024
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 13, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
docutils Tags upstream Docutils issues internals:typing type:enhancement enhance or introduce a new feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants