-
Notifications
You must be signed in to change notification settings - Fork 97
Fix docs #457
Fix docs #457
Conversation
Codecov Report
@@ Coverage Diff @@
## master #457 +/- ##
==========================================
+ Coverage 94.27% 94.66% +0.39%
==========================================
Files 150 150
Lines 8782 9649 +867
Branches 711 726 +15
==========================================
+ Hits 8279 9134 +855
- Misses 503 515 +12
Continue to review full report at Codecov.
|
Codecov Report
@@ Coverage Diff @@
## master #457 +/- ##
=========================================
+ Coverage 94.27% 94.67% +0.4%
=========================================
Files 150 150
Lines 8782 9724 +942
Branches 711 726 +15
=========================================
+ Hits 8279 9206 +927
- Misses 503 518 +15
Continue to review full report at Codecov.
|
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.
On minor nit on a comment, but this is a great cleanup!
import {BucketBoundaries} from './bucket-boundaries'; | ||
import {MetricUtils} from './metric-utils'; | ||
import {Recorder} from './recorder'; | ||
import {AggregationData, AggregationType, Measure, Measurement, StatsExemplar, View} from './types'; | ||
|
||
const RECORD_SEPARATOR = String.fromCharCode(30); | ||
|
||
/** Implementation for the Base View class. */ |
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.
Could this comment be more descriptive in terms of the role that BaseView
plays?
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.
Done in be531ae. Thanks
No description provided.