search

BetterWorks / django-anonymizer

MIT License
14 stars 12 forks source link

speed improvements, bug fixes, & updates #29

Closed terite closed 7 years ago

codecov-io commented 7 years ago

Codecov Report

Merging #29 into master will decrease coverage by 0.65%. The diff coverage is 33.33%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #29      +/-   ##
==========================================
- Coverage   76.64%   75.98%   -0.66%     
==========================================
  Files           7        7              
  Lines         381      379       -2     
  Branches       65       66       +1     
==========================================
- Hits          292      288       -4     
+ Misses         78       76       -2     
- Partials       11       15       +4
Impacted Files Coverage Δ
anonymizer/base.py 81.81% <33.33%> (-1.07%) :arrow_down:

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 32e5f44...43897f2. Read the comment docs.

danielcmartin commented 7 years ago

where does the speed improvement come from? the short circuit logic in base.py?

also what is the bug fix?