-
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
Merge upstream updates #69
Conversation
That should create packages for stable releases on tag event
Add simple test that ensure behavior of different methods is as in description
graphite-web apparantly allow you to escape special reserved characters in metric names Modify parser to allow to escape special characters as well That should fix go-graphite#661
…aphite#678) * some benchmarks for functions * extend function description with name/value change * extend tests for groupByNode/aliasByNode * cleanup ExtractMetric from = parse * refactor for aliasByNode/groupByNode * refactor for aliasByMetric * refactor for aliasByBase64 * verify tags in fuctions tests * refactor for groupByTags * fixes for absolute function * fix for aggregate function * fixes for aggregateLine function * fixes for alias function * modify function description * fix for aliasSub function * extend tests/add bencmarks for other functions * refactor for asPercent function * revert ExtractMetric * refactor aliasByRedis/aliasByPostgres functions * refactor aliasByNode (remove ExtractMetric usage) * refactor store name tag * refactor for other functions * fix for divideSeries function * fix for transformNull function * refactor and fixes for functions
* Update README to reflect current supported OSs, Architectures and Golang versions * Drop Golang prior to 1.18 from tests
Migrate it to a proper merry v2 style of error handling, that should produce better error messages in case of any failure that might happen.
|
This comment has been minimized.
This comment has been minimized.
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
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
Go coverage report: Click to expand.
Go lint report: No issues found. 😎 |
The merge conflicts resolving was reviewed here #62
Here I compressed all those changes into the Merge commit.