Skip to content
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

Speed up deleted item check #106

Merged
merged 2 commits into from
Aug 20, 2018
Merged

Speed up deleted item check #106

merged 2 commits into from
Aug 20, 2018

Conversation

LegoStormtroopr
Copy link
Contributor

Looks like a few people have had this idea, but I got the PR first.
And it has a bumped version number

@codecov
Copy link

codecov bot commented Jul 19, 2018

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #106   +/-   ##
=======================================
  Coverage   78.52%   78.52%           
=======================================
  Files           4        4           
  Lines         433      433           
  Branches      104      103    -1     
=======================================
  Hits          340      340           
  Misses         70       70           
  Partials       23       23
Impacted Files Coverage Δ
reversion_compare/compare.py 85.99% <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 3179a76...f4ee949. Read the comment docs.

@coveralls
Copy link

coveralls commented Jul 19, 2018

Coverage Status

Coverage decreased (-0.1%) to 85.017% when pulling f4ee949 on LegoStormtroopr:master into 3179a76 on jedie:master.

@LegoStormtroopr
Copy link
Contributor Author

This would fix #95 and a few others have had a go at fixing this as well in their own branches.

Its only failed based on a tiny coverage decrease. Any change this can get pulled in?

@jedie jedie merged commit cd13f57 into jedie:master Aug 20, 2018
@jedie
Copy link
Owner

jedie commented Aug 20, 2018

Tiny coverage decrease is not really a Problem ;)

@jedie
Copy link
Owner

jedie commented Sep 13, 2018

It's now release as v0.8.5

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