-
Notifications
You must be signed in to change notification settings - Fork 5
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
DOCS 1484: Update DB Extractor schema JSON #290
Conversation
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## master #290 +/- ##
==========================================
- Coverage 79.56% 79.53% -0.03%
==========================================
Files 22 22
Lines 2001 2003 +2
==========================================
+ Hits 1592 1593 +1
- Misses 409 410 +1 |
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!
Some comments, mostly nitpicks
schema/metrics_config.schema.json
Outdated
@@ -57,36 +57,36 @@ | |||
}, | |||
"asset-name": { | |||
"type": "string", | |||
"description": "Enter the name for a CDF asset that will have all the metrics time series attached to it." | |||
"description": "Enter the name for a CDF asset with all the metrics time series attached to it." |
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.
The asset might not yet exist, we will set all the time series on it, which why it was written in the tense it was
Co-authored-by: Mathias Lohne <[email protected]>
Co-authored-by: Mathias Lohne <[email protected]>
Co-authored-by: Mathias Lohne <[email protected]>
Co-authored-by: Mathias Lohne <[email protected]>
No description provided.