Skip to content
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

Fix grammatical number of metric semconv units #3298

Merged
merged 8 commits into from
Mar 8, 2023

Conversation

MrAlias
Copy link
Contributor

@MrAlias MrAlias commented Mar 6, 2023

Fixes #3295

Changes

Currently the metric semantic conventions are defined with non-units of improper grammatical number. This update changes existing plurals into singulars (I didn't find any instances of singulars that needed to be switched to plurals with a quantity).

@MrAlias MrAlias added area:semantic-conventions Related to semantic conventions spec:metrics Related to the specification/metrics directory labels Mar 6, 2023
@MrAlias MrAlias requested review from a team as code owners March 6, 2023 22:17
@reyang reyang merged commit 0b87a95 into open-telemetry:main Mar 8, 2023
@MrAlias MrAlias deleted the unit-gramatic-number branch March 8, 2023 17:18
lmolkova pushed a commit to lmolkova/opentelemetry-specification that referenced this pull request Mar 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:semantic-conventions Related to semantic conventions spec:metrics Related to the specification/metrics directory
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Define arbitrary metric units singularly
5 participants