Repository navigation
Conversation
|
pyrefly now crashes with "Stdlib missing class |
This comment has been minimized.
This comment has been minimized.
|
Removing it is probably the wrong approach as primer shows, and since we're likely just going to ignore that error code anyway. But I'll try something different. |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
|
Let's wait for #16587, which should help with the primer hits. |
_IOBase.read() and write() into functions
This comment has been minimized.
This comment has been minimized.
|
According to mypy_primer, this change has no effect on the checked open source code. 🤖🎉 |
|
The ty error in the PR description seems like a bug? A |
Probably, but I think this PR is an improvement in any case to model the virtual methods as such, especially with the arguments being modelled now. |
Originally introduced in #4912, because the docs say:
In the PR I said:
This now seems to be the case, since ty 0.0.87 not very unreasonably complains in sub-classes (see #16551):
Since the interface may vary according to the docs, I'm not sure how useful there fields are in any case. Let's first see what primer says.