Skip to content
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

Add py.typed for mypy #248

Merged
merged 1 commit into from
Aug 18, 2022
Merged

Add py.typed for mypy #248

merged 1 commit into from
Aug 18, 2022

Conversation

pjbull
Copy link
Member

@pjbull pjbull commented Aug 17, 2022

Manually confirmed mypy fails on script with latest PyPI version, but succeeds with whl/source package built with this change.

Closes #243

@github-actions
Copy link
Contributor

@codecov-commenter
Copy link

codecov-commenter commented Aug 17, 2022

Codecov Report

Merging #248 (ce5a598) into master (4cc0d33) will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master    #248   +/-   ##
======================================
  Coverage    95.0%   95.0%           
======================================
  Files          21      21           
  Lines        1324    1324           
======================================
  Hits         1258    1258           
  Misses         66      66           

@jayqi jayqi self-requested a review August 18, 2022 15:35
@pjbull pjbull merged commit 71f8751 into master Aug 18, 2022
@pjbull pjbull deleted the 243-mypy-typed branch August 18, 2022 15:48
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.

Can't use cloudpathlib with mypy
3 participants