-
Notifications
You must be signed in to change notification settings - Fork 1
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
Upstream sync #96
Upstream sync #96
Conversation
* fix for extract tags in functions (tags by default are inherited from first metric) * update docs * cleanup deprecated function
|
Go coverage report: Click to expand.
Go lint report: No issues found. 😎 |
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.
LGTM
It seems like the GitHub action is not working, so doing the merge manually.
The merging went pretty smooth apart from conflicts in the
AggregateSeries
function, in which both the upstream and us added an extra arg.