Skip to content

More dynamic approach to dict views types#435

Open
brl0 wants to merge 13 commits into
cloudpipe:masterfrom
brl0:views_types
Open

More dynamic approach to dict views types#435
brl0 wants to merge 13 commits into
cloudpipe:masterfrom
brl0:views_types

Conversation

@brl0

@brl0 brl0 commented Aug 8, 2021

Copy link
Copy Markdown
Contributor

This is a revisit of my previous effort to add support for dict views type objects. I had some ideas rattling around to try to make it a bit more dynamic and extensible.

@codecov

codecov Bot commented Aug 8, 2021

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 61.66667% with 23 lines in your changes missing coverage. Please review.
✅ Project coverage is 89.55%. Comparing base (343da11) to head (513a938).
⚠️ Report is 76 commits behind head on master.

Files with missing lines Patch % Lines
cloudpickle/cloudpickle_fast.py 64.86% 7 Missing and 6 partials ⚠️
cloudpickle/cloudpickle.py 56.52% 8 Missing and 2 partials ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #435      +/-   ##
==========================================
- Coverage   92.36%   89.55%   -2.81%     
==========================================
  Files           4        4              
  Lines         720      747      +27     
  Branches      150      158       +8     
==========================================
+ Hits          665      669       +4     
- Misses         34       49      +15     
- Partials       21       29       +8     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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.

1 participant