Skip to content

Conversation

@mttbernardini
Copy link
Contributor

No description provided.

@glyph
Copy link
Owner

glyph commented Feb 13, 2025

Thank you @mttbernardini for your attention to detail!

This commit was sponsored by Matt Campbell, Blaise Pabon, Cassie
Jones, and my other patrons.  If you want to join them, you can
support my work at https://glyph.im/patrons/.
This commit was sponsored by Blaise Pabon, Matt Campbell, and my other
patrons.  If you want to join them, you can support my work at
https://glyph.im/patrons/.
c.f. python/mypy#4717

This commit was sponsored by Blaise Pabon, Matt Campbell, and my other
patrons.  If you want to join them, you can support my work at
https://glyph.im/patrons/.
Copy link
Owner

@glyph glyph left a comment

Choose a reason for hiding this comment

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

Obviously the fix here is correct, so I will address the testing issues, which I think are incidental.

This commit was sponsored by Blaise Pabon, and my other patrons.  If
you want to join them, you can support my work at
https://glyph.im/patrons/.
This commit was sponsored by Blaise Pabon, and my other patrons.  If
you want to join them, you can support my work at
https://glyph.im/patrons/.
This commit was sponsored by Blaise Pabon, and my other patrons.  If
you want to join them, you can support my work at
https://glyph.im/patrons/.
@glyph glyph merged commit d1666ed into glyph:trunk Feb 14, 2025
11 checks passed
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