search

cloudfoundry-attic / cf-abacus

CF usage metering and aggregation
Apache License 2.0
98 stars 86 forks source link

fix(reporting) use end time of aggregated doc for bounds calculation #1154

Closed vgpetkov closed 5 years ago

cfdreddbot commented 5 years ago

:white_check_mark: Hey vgpetkov! The commit authors and yourself have already signed the CLA.

cf-gitbot commented 5 years ago

We have created an issue in Pivotal Tracker to manage this:

https://www.pivotaltracker.com/story/show/163112136

The labels on this github issue will be updated when the story is started.

codecov[bot] commented 5 years ago

Codecov Report

Merging #1154 into master will not change coverage. The diff coverage is 100%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1154   +/-   ##
=======================================
  Coverage   94.88%   94.88%           
=======================================
  Files         128      128           
  Lines        3477     3477           
=======================================
  Hits         3299     3299           
  Misses        178      178
Impacted Files Coverage Δ
lib/aggregation/reporting/src/lib/summarizer.js 89.23% <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 98b4b8f...87b0c82. Read the comment docs.