Closed dlebech closed 7 years ago
Merging #18 into master will not change coverage. The diff coverage is
100%
.
@@ Coverage Diff @@
## master #18 +/- ##
=====================================
Coverage 100% 100%
=====================================
Files 5 5
Lines 232 267 +35
Branches 55 64 +9
=====================================
+ Hits 232 267 +35
Impacted Files | Coverage Δ | |
---|---|---|
lib/counter.js | 100% <100%> (ø) |
:arrow_up: |
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 993a297...d8ebcb3. Read the comment docs.
This PR adds a utility function to the timestamped counter that can trim the set of event objects stored for the counter. This is useful if the number of event objects have gotten very large, but it is not generally a good idea to use the function very often, because it naively trims values without considering the impact to reporting.