Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Make --cache-fine-grained imply --local-partial-types - #10737

Merged
JukkaL merged 2 commits into
masterfrom
local_partial
Jun 30, 2021
Merged

Make --cache-fine-grained imply --local-partial-types#10737
JukkaL merged 2 commits into
masterfrom
local_partial

Conversation

@msullivan

Copy link
Copy Markdown
Collaborator

dmypy requires local-partial-types, so without specifying it, we generate
a cache that is useless to dmypy.

Prompted by #10735.

Test Plan

Tested manually generating a cache for mypy and then checking it with dmypy.

dmypy requires local-partial-types, so without specifying it, we generate
a cache that is useless to dmypy.

Prompted by #10735.
@github-actions

This comment has been minimized.

@github-actions

Copy link
Copy Markdown
Contributor

Diff from mypy_primer, showing the effect of this PR on open source code:

schemathesis (https://github.com/schemathesis/schemathesis.git)
+ src/schemathesis/specs/openapi/negative/__init__.py: note: At top level:
+ src/schemathesis/specs/openapi/negative/__init__.py:10: error: Need type annotation for "_VALIDATORS_CACHE" (hint: "_VALIDATORS_CACHE: Dict[<type>, <type>] = ...")
+ src/schemathesis/serializers.py:23: error: Need type annotation for "SERIALIZERS" (hint: "SERIALIZERS: Dict[<type>, <type>] = ...")
- src/schemathesis/models.py:789: error: Unexpected keyword argument "http_version" for "Response"

@JukkaL JukkaL left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thanks!

@JukkaL
JukkaL merged commit 1985928 into master Jun 30, 2021
@JukkaL
JukkaL deleted the local_partial branch June 30, 2021 12:38
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