Skip to content

Conversation

godardth
Copy link

Directly import six library as it is not included anymore in Django utils.

@codecov
Copy link

codecov bot commented Dec 31, 2019

Codecov Report

Merging #77 into master will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #77   +/-   ##
=======================================
  Coverage   64.91%   64.91%           
=======================================
  Files          12       12           
  Lines         761      761           
=======================================
  Hits          494      494           
  Misses        267      267
Impacted Files Coverage Δ
polymorphic_tree/models.py 63.24% <100%> (ø) ⬆️

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 479934a...8c94501. Read the comment docs.

@vdboor
Copy link
Member

vdboor commented Jan 4, 2020

Thanks for pointing this out! I've used your PR as a basis. The commit uses the future.utils library instead of adding six as it already contained the required functions :-)

@vdboor vdboor closed this Jan 4, 2020
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.

2 participants