Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings
This repository was archived by the owner on Nov 7, 2024. It is now read-only.

Export typing information #923

Open
JanLuca wants to merge 2 commits into google:master
base: master
Choose a base branch
Loading
from JanLuca:export_typing
Open

Conversation

@JanLuca
Copy link

@JanLuca JanLuca commented May 20, 2021

This commit adds the flag that the package contains typing
information. Therefore other packages can use the typing data to check their
code by mypy.

Ref.: https://mypy.readthedocs.io/en/stable/installed_packages.html#making-pep-561-compatible-packages

Signed-off-by: Jan Luca Naumann j.naumann@fu-berlin.de

This commit adds the flag that the package contains typing
information. Therefore other packages can use the typing data to check their
code by mypy.
Ref.: https://mypy.readthedocs.io/en/stable/installed_packages.html#making-pep-561-compatible-packages
Signed-off-by: Jan Luca Naumann <j.naumann@fu-berlin.de>
Copy link

google-cla bot commented May 20, 2021

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

📝 Please visit https://cla.developers.google.com/ to sign.

Once you've signed (or fixed any issues), please reply here with @googlebot I signed it! and we'll verify it.


What to do if you already signed the CLA

Individual signers
Corporate signers

i️ Googlers: Go here for more info.

Copy link
Author

JanLuca commented May 20, 2021

@googlebot I signed it!

Copy link
Contributor

@alewis alewis left a comment

Choose a reason for hiding this comment

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

Looks good!

Copy link
Contributor

mganahl commented May 31, 2021

Thanks for the PR! I'll fix the build and merge this.

Copy link

Codecov Report

Merging #923 (357f430) into master (20dc783) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@ Coverage Diff @@
## master #923 +/- ##
=======================================
 Coverage 98.04% 98.04% 
=======================================
 Files 129 129 
 Lines 22394 22394 
=======================================
 Hits 21957 21957 
 Misses 437 437 
Impacted Files Coverage Δ
setup.py 0.00% <ø> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 20dc783...357f430. Read the comment docs.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

1 more reviewer

@alewis alewis alewis approved these changes

Reviewers whose approvals may not affect merge requirements

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

AltStyle によって変換されたページ (->オリジナル) /