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

Remove DetectMimeTypeTika and related test. #331

Open
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
3 participants
@ruebot
Copy link
Member

commented Jul 25, 2019

GitHub issue(s): #330

What does this Pull Request do?

Remove DetectMimeTypeTika and related test. See #330 for background and any discussion.

How should this be tested?

@ianmilligan1 would you mind putting this PR through a barrage of aut tests? Might as well take advantage of your offer, and it'll give us more data for discussion on the issue 😄

@ruebot

This comment has been minimized.

Copy link
Member Author

commented Jul 25, 2019

☠️ code reaper ☠️

@ianmilligan1

This comment has been minimized.

Copy link
Member

commented Jul 25, 2019

Sure thing! Am in online course filming all morning but should be able to do this in the afternoon.

@ruebot

This comment has been minimized.

Copy link
Member Author

commented Jul 25, 2019

Cool. No rush on this one.

@codecov-io

This comment has been minimized.

Copy link

commented Jul 25, 2019

Codecov Report

Merging #331 into master will decrease coverage by 0.17%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #331      +/-   ##
==========================================
- Coverage   74.97%   74.79%   -0.18%     
==========================================
  Files          39       38       -1     
  Lines        1123     1115       -8     
  Branches      197      196       -1     
==========================================
- Hits          842      834       -8     
  Misses        215      215              
  Partials       66       66

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 19b49e1...a4d68b0. Read the comment docs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.