Skip to content

Commit

Permalink
Merge pull request #740 from dashpole/update_om_links
Browse files Browse the repository at this point in the history
update links to openmetrics to reference the v1.0.0 release
  • Loading branch information
ArthurSens authored Dec 14, 2024
2 parents d88ee1f + e3926e2 commit 0a89b98
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions expfmt/openmetrics_create.go
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,7 @@ type EncoderOption func(*encoderOption)

// WithCreatedLines is an EncoderOption that configures the OpenMetrics encoder
// to include _created lines (See
// https://github.com/OpenObservability/OpenMetrics/blob/main/specification/OpenMetrics.md#counter-1).
// https://github.com/prometheus/OpenMetrics/blob/v1.0.0/specification/OpenMetrics.md#counter-1).
// Created timestamps can improve the accuracy of series reset detection, but
// come with a bandwidth cost.
//
Expand Down Expand Up @@ -102,7 +102,7 @@ func WithUnit() EncoderOption {
//
// - According to the OM specs, the `# UNIT` line is optional, but if populated,
// the unit has to be present in the metric name as its suffix:
// (see https://github.com/OpenObservability/OpenMetrics/blob/main/specification/OpenMetrics.md#unit).
// (see https://github.com/prometheus/OpenMetrics/blob/v1.0.0/specification/OpenMetrics.md#unit).
// However, in order to accommodate any potential scenario where such a change in the
// metric name is not desirable, the users are here given the choice of either explicitly
// opt in, in case they wish for the unit to be included in the output AND in the metric name
Expand Down

0 comments on commit 0a89b98

Please sign in to comment.