Skip to content

Commit

Permalink
Fixed conditionally_required requirement_level. (#201)
Browse files Browse the repository at this point in the history
  • Loading branch information
jerbly committed Sep 11, 2023
1 parent 0546bb1 commit 870e84d
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 2 deletions.
2 changes: 2 additions & 0 deletions semantic-conventions/CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,8 @@ Please update the changelog as part of any significant pull request.

- Render template-type attributes from yaml files
([#186](https://github.com/open-telemetry/build-tools/pull/186))
- Fix conditionally_required definition in semconv.schema.json
([#201](https://github.com/open-telemetry/build-tools/pull/201))

## v0.20.0

Expand Down
4 changes: 2 additions & 2 deletions semantic-conventions/semconv.schema.json
Original file line number Diff line number Diff line change
Expand Up @@ -349,7 +349,7 @@
{
"properties": {
"requirement_level": {
"description": "specifies the attribute requirement level. Can be 'required', 'conditionally_required', 'recommended', or 'opt_in'. When omitted, the attribute is 'recommended'. When set to 'conditionally_required', the string provided as <condition> MUST specify the conditions under which the attribute is required.",
"description": "specifies the attribute requirement level. Can be 'required', 'conditionally_required', 'recommended', or 'opt_in'. When omitted, the attribute is 'recommended'. When set to 'conditionally_required', the string provided MUST specify the conditions under which the attribute is required.",
"oneOf": [
{
"const": "required"
Expand All @@ -361,7 +361,7 @@
"conditionally_required"
],
"properties": {
"condition": {
"conditionally_required": {
"type": "string"
}
}
Expand Down

0 comments on commit 870e84d

Please sign in to comment.