Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

dev(hansbug): add popitem method for TreeValue #52

Merged
merged 2 commits into from
Jul 6, 2022
Merged

Conversation

HansBug
Copy link
Member

@HansBug HansBug commented Jul 6, 2022

#44

@HansBug HansBug added the enhancement New feature or request label Jul 6, 2022
@HansBug HansBug added this to the v1.4.0 Release milestone Jul 6, 2022
@HansBug HansBug self-assigned this Jul 6, 2022
@codecov
Copy link

codecov bot commented Jul 6, 2022

Codecov Report

Merging #52 (5c6ea4d) into main (17cdc03) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main      #52   +/-   ##
=======================================
  Coverage   99.24%   99.25%           
=======================================
  Files          35       35           
  Lines        2126     2135    +9     
=======================================
+ Hits         2110     2119    +9     
  Misses         16       16           
Flag Coverage Δ
unittests 99.25% <100.00%> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
treevalue/tree/common/storage.pyx 100.00% <100.00%> (ø)
treevalue/tree/tree/tree.pyx 100.00% <100.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 17cdc03...5c6ea4d. Read the comment docs.

@HansBug HansBug merged commit a144b91 into main Jul 6, 2022
@HansBug HansBug deleted the dev/mutablemapping branch July 6, 2022 14:31
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant