-
Notifications
You must be signed in to change notification settings - Fork 0
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
BAC-296: update Client.Evaluate
call to version 1.2alpha1 (breaking)
#34
Conversation
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.
Ensure you enable an automatic formatter.
thank you sindre, i'll be back with something corrected |
947ad27
to
1058855
Compare
495e64a
to
e324b40
Compare
e324b40
to
4c9a9c4
Compare
Split in two documents, but try to follow this commit-style: E.g.
In PR title, add issue prefix: For this case, there is no (sub-component) prefix needed. |
ad74157
to
5e45f98
Compare
Evaluate
call to version 1.2alpha1 (breaking)
Evaluate
call to version 1.2alpha1 (breaking)Client.Evaluate
call to version 1.2alpha1 (breaking)
5e45f98
to
2e9cc8a
Compare
this introduces a breaking change in `Evaluate` requiring users of this function to update as follows: ``` Evaluate(items, calculations, data) ↓ Evaluate(data). Items(items...). Calculations(calculations...) ``` the type `AggregateMethod` has been updated as follows: ``` AggregateMethod → TimeAggregationMethod ``` with new value names: ``` AggregateDefault → TimeAggregationDefault AggregateCount → TimeAggregationCount AggregateMin → TimeAggregationMin AggregateMax → TimeAggregationMax AggregateSum → TimeAggregationSum AggregateAvg → TimeAggregationAvg AggregateStateHistSeconds → TimeAggregationSeconds AggregateStateHistPercent → TimeAggregationPercent AggregateStateHistRate → TimeAggregationRate ``` the type `ItemAggregation` has been updated as follows: ``` ItemAggregation → EvaluateItem ```
2e9cc8a
to
b0b1451
Compare
i have updated the clarify go sdk to support the new
evaluate
parametergroups
. i have also updated some names to match the changes introduced in BAC-282. lastly i have updated theautomation
andexamples
packages to use the new builder styleEvaluateRequest
type