Skip to content

Mappingproxy #905

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

Merged
merged 2 commits into from
May 3, 2019
Merged

Mappingproxy #905

merged 2 commits into from
May 3, 2019

Conversation

cthulahoops
Copy link
Collaborator

No description provided.

@codecov-io
Copy link

Codecov Report

Merging #905 into master will increase coverage by 0.02%.
The diff coverage is 73.52%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #905      +/-   ##
==========================================
+ Coverage   63.93%   63.96%   +0.02%     
==========================================
  Files          88       89       +1     
  Lines       14922    14955      +33     
  Branches     3335     3337       +2     
==========================================
+ Hits         9541     9566      +25     
- Misses       3175     3182       +7     
- Partials     2206     2207       +1
Impacted Files Coverage Δ
vm/src/obj/objobject.rs 76.47% <0%> (ø) ⬆️
vm/src/obj/objtype.rs 72.58% <50%> (-0.72%) ⬇️
vm/src/pyobject.rs 79.05% <75%> (-0.03%) ⬇️
vm/src/obj/objmappingproxy.rs 85.71% <85.71%> (ø)

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 8f3b62b...65d6025. Read the comment docs.

@windelbouwman
Copy link
Contributor

Looks good to me!

@cthulahoops cthulahoops merged commit 1d6e8d5 into master May 3, 2019
@sanxiyn sanxiyn mentioned this pull request May 9, 2019
@windelbouwman windelbouwman deleted the mappingproxy branch July 6, 2019 09:00
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.

3 participants