Skip to content

Conversation

srittau
Copy link
Member

@srittau srittau commented Sep 30, 2025

Also remove --no-pyi-aware-file-checker option

Closes: #522

Also remove --no-pyi-aware-file-checker option

Closes: PyCQA#522

This comment has been minimized.

This comment has been minimized.

Copy link
Collaborator

@AlexWaygood AlexWaygood left a comment

Choose a reason for hiding this comment

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

Thank you!

This comment has been minimized.

This comment has been minimized.

Copy link
Collaborator

@AlexWaygood AlexWaygood left a comment

Choose a reason for hiding this comment

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

Thank you!


@staticmethod
def add_options(parser: OptionManager) -> None:
"""This is brittle, there's multiple levels of caching of defaults."""
Copy link
Collaborator

Choose a reason for hiding this comment

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

I sort of worry that nobody on the current maintainers team knows what this cryptic comment is actually referring to ¯\_(ツ)_/¯

Copy link
Member Author

Choose a reason for hiding this comment

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

... and it's gone.

Copy link
Member Author

Choose a reason for hiding this comment

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

(Unless @ambv remembers what he meant with that comment. It's only been 10 years, after all.)

Copy link

This change has no effect on typeshed. 🤖🎉

@srittau srittau merged commit 328f342 into PyCQA:main Sep 30, 2025
23 checks passed
@srittau srittau deleted the rm-f821-monkeypatch branch September 30, 2025 13:20
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.

Remove F821 monkeypatch

2 participants