Skip to content

Conversation

@kurtmckee
Copy link
Contributor

Recent example of a mypy CI failure:

readme_renderer/rst.py:26: error: Unused "type: ignore" comment  [unused-ignore]
readme_renderer/rst.py:33: error: Argument 1 of "emptytag" is incompatible with supertype "docutils.writers._html_base.HTMLTranslator"; supertype defines the argument type as "Element"  [override]
readme_renderer/rst.py:33: note: This violates the Liskov substitution principle
readme_renderer/rst.py:33: note: See https://mypy.readthedocs.io/en/stable/common_issues.html#incompatible-overrides

The CI failures appear to be related to an updated release of types-docutils.

The failures were:

```
readme_renderer/rst.py:26: error: Unused "type: ignore" comment  [unused-ignore]
readme_renderer/rst.py:33: error: Argument 1 of "emptytag" is incompatible with supertype "docutils.writers._html_base.HTMLTranslator"; supertype defines the argument type as "Element"  [override]
readme_renderer/rst.py:33: note: This violates the Liskov substitution principle
readme_renderer/rst.py:33: note: See https://mypy.readthedocs.io/en/stable/common_issues.html#incompatible-overrides
```

The CI failures appear to be related to an updated release of `types-docutils`.
@kurtmckee
Copy link
Contributor Author

Well...shoot fire. The mypy tests now pass, but an unrelated issue has cropped up, and as with the mypy failures, the new issues are occurring in main with no code changes.

Copy link
Member

@miketheman miketheman left a comment

Choose a reason for hiding this comment

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

I've confirmed locally that when combined with the test changes, everything passes to green.

In order to merge the two PRs, I'll override the allowance for the project to force this through, to be quickly followed by the test changes.

@miketheman miketheman merged commit 9d94848 into pypa:main Oct 9, 2025
2 of 9 checks passed
@kurtmckee kurtmckee deleted the fix-mypy branch October 10, 2025 13:41
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.

2 participants