Skip to content

Conversation

roidelapluie
Copy link
Member

This PR adds an Experimental flag to the functions.

This can be used by #13059 but also xrate and other future functions.

@roidelapluie roidelapluie force-pushed the experimentalflag branch 2 times, most recently from 4e68b01 to 5a138b5 Compare November 8, 2023 03:03
@roidelapluie roidelapluie requested a review from SuperQ November 8, 2023 16:32
@@ -22,7 +22,8 @@ import (
"github.com/prometheus/prometheus/model/labels"
"github.com/prometheus/prometheus/model/value"
"github.com/prometheus/prometheus/model/histogram"
"github.com/prometheus/prometheus/promql/parser/posrange"

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This blank line causes a number of extra diffs in the generated code; is it helpful?

@juliusv
Copy link
Member

juliusv commented Nov 14, 2023

👍 besides @bboreham's comment.

This PR adds an Experimental flag to the functions.

This can be used by prometheus#13059
but also xrate and other future functions.

Signed-off-by: Julien Pivotto <roidelapluie@o11y.eu>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants