-
-
Notifications
You must be signed in to change notification settings - Fork 26
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
🧹 some necessary maintenance #80
Conversation
Codecov Report
@@ Coverage Diff @@
## main #80 +/- ##
=======================================
Coverage 97.91% 97.91%
=======================================
Files 23 23
Lines 1245 1245
=======================================
Hits 1219 1219
Misses 26 26
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
Ready for review @jonasvdd |
I will review it this evening! 😄 |
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.
LGTM! nice cleanup!
@jonavdd what would you change to the logger of the |
Maybe also show the duration in |
Sure will look into it and submit a new PR! |
tsfresh
(dev) dependency, fixing the issue with installingmatrixprofile
Add matrix profile as an optional import. blue-yonder/tsfresh#985README.md
Makefile
CONTRIBUTING.md