Add history-rm command#1128
Conversation
|
Looks good. There's only a warning from the compiler when compiling the test. Could you fix that? |
Should be fixed now |
|
You have some memory leaks, probably in the tests. You can check this locally with |
|
It's fixed |
|
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #1128 +/- ##
==========================================
+ Coverage 65.26% 65.37% +0.11%
==========================================
Files 46 46
Lines 7358 7420 +62
==========================================
+ Hits 4802 4851 +49
- Misses 2556 2569 +13
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
Merged, thank you! |
Implements #308