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 PEP 561 compatibility #404

Open
wants to merge 5 commits into
base: master
from
Open

Add PEP 561 compatibility #404

wants to merge 5 commits into from

Conversation

@hoefling
Copy link

@hoefling hoefling commented Feb 7, 2020

This PR adds PEP 561 compatibility to msgpack.

I keep the type stubs from this PR for development with msgpack. The idea is to make them available to everyone by adding them to the distribution. There are no changes to the msgpack code itself, the stub is completely separated from the real code.

If you are against adding type stubs, I'd still be grateful for a review that would improve the stub quality - maybe you have a good advice on how to deal with the remaining Any types.

@hoefling hoefling requested a review from methane Feb 7, 2020
@codecov-io
Copy link

@codecov-io codecov-io commented Feb 7, 2020

Codecov Report

Merging #404 into master will decrease coverage by 0.11%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #404      +/-   ##
==========================================
- Coverage   88.06%   87.94%   -0.12%     
==========================================
  Files           5        5              
  Lines         846      846              
==========================================
- Hits          745      744       -1     
- Misses        101      102       +1
Impacted Files Coverage Δ
msgpack/fallback.py 88.81% <0%> (-0.14%) ⬇️

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 2849f55...c7f6c8b. Read the comment docs.

hoefling added 5 commits Feb 7, 2020
Signed-off-by: Oleg Höfling <oleg.hoefling@gmail.com>
Signed-off-by: Oleg Höfling <oleg.hoefling@gmail.com>
Signed-off-by: Oleg Höfling <oleg.hoefling@gmail.com>
…a mypy run

Signed-off-by: Oleg Höfling <oleg.hoefling@gmail.com>
Signed-off-by: Oleg Höfling <oleg.hoefling@gmail.com>
@hoefling hoefling force-pushed the hoefling:type-stubs branch from c36cc1e to c7f6c8b Feb 21, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.