-
Notifications
You must be signed in to change notification settings - Fork 3.7k
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
feat: add pruning config to toml #5090
feat: add pruning config to toml #5090
Conversation
There don't seem to be tests related to pruning? |
Codecov Report
@@ Coverage Diff @@
## master #5090 +/- ##
==========================================
+ Coverage 54.97% 54.98% +0.01%
==========================================
Files 297 297
Lines 18224 18225 +1
==========================================
+ Hits 10018 10021 +3
+ Misses 7468 7466 -2
Partials 738 738 |
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.
Thanks @mkg20001 👍
Co-Authored-By: Alexander Bezobchuk <[email protected]>
Co-Authored-By: Alexander Bezobchuk <[email protected]>
b0e897c
to
cbe9f23
Compare
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.
ACK 🎉
Co-Authored-By: Alexander Bezobchuk <[email protected]>
Fixes #4215
Targeted PR against correct branch (see CONTRIBUTING.md)
Linked to github-issue with discussion and accepted design OR link to spec that describes this work.
Wrote tests
Updated relevant documentation (
docs/
)Added a relevant changelog entry to the
Unreleased
section inCHANGELOG.md
Re-reviewed
Files changed
in the github PR explorerFor Admin Use: