-
Notifications
You must be signed in to change notification settings - Fork 87
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 unused apply.py file #449
Conversation
Codecov Report
@@ Coverage Diff @@
## master #449 +/- ##
==========================================
+ Coverage 97.47% 98.19% +0.71%
==========================================
Files 105 104 -1
Lines 3292 3260 -32
==========================================
- Hits 3209 3201 -8
+ Misses 83 59 -24 Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🍶
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 lol awesome! So that was all dead code? I wonder if our code coverage report can show us if there's any other stuff like that.
Also thanks to @jeremyliweishih today I learned there's a sake emoji 😂 |
@dsherry Yup, seemed like it was code from the very beginnings of evalml 😆 If you go to https://codecov.io/gh/FeatureLabs/evalml, you can see the sunburst of overall code coverage--this file is red because there is none. Not sure if there's a better visualization but that's something! :D |
@angela97lin ooh cool, thanks! |
Closes #448.