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

Closed
wants to merge 1 commit into from

Conversation

@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.

@ianmilligan1
Copy link
Member

left a comment

Finished early!

Ran through the suite of scripts in our documentation and everything worked on the example data file. So it's good to go on my end.

@ruebot

This comment has been minimized.

Copy link
Member Author

commented Jul 31, 2019

See #330 and #302.

@ruebot ruebot closed this Jul 31, 2019

@ruebot ruebot deleted the issue-330 branch Jul 31, 2019

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