Skip to content

Fix stubtest for third-party mock package #6685

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

Merged
merged 1 commit into from
Dec 25, 2021
Merged

Conversation

srittau
Copy link
Collaborator

@srittau srittau commented Dec 24, 2021

Removed a few "Any" annotations that could be typed. (There are more.) Add
package to pyrightconfig.stricter.json ignore list.

Also removed a few unused private classes and methods. They can be re-added
if a user reports them missing.

Removed a few "Any" annotations that could be typed. (There are more.) Add
package to pyrightconfig.stricter.json ignore list.

Also removed a few unused private classes and methods. They can be re-added
if a user reports them missing.
@srittau srittau linked an issue Dec 24, 2021 that may be closed by this pull request
# This overload also covers the case, where new==DEFAULT. In this case, self is _patch[Any].
# Ideally we'd be able to add an overload for it so that self is _patch[MagicMock],
# but that's impossible with the current type system.
@overload
Copy link
Collaborator

Choose a reason for hiding this comment

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

Why was the first overload removed?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

new is a mandatory argument.

@Akuli Akuli merged commit 15958a2 into python:master Dec 25, 2021
@srittau srittau deleted the fix-mock branch December 25, 2021 09:46
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.

stubtest is failing for mock
2 participants