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

Nulls SHOULD NOT be allowed in arrays. #1214

Merged
merged 3 commits into from
Nov 11, 2020

Conversation

Oberon00
Copy link
Member

@Oberon00 Oberon00 commented Nov 9, 2020

Fixes #1213, closes #1155.

Changes

Clarify null SHOULD NOT be allowed even in arrays.

CC @anuraaga

@Oberon00 Oberon00 added area:api Cross language API specification issue spec:resource Related to the specification/resource directory spec:trace Related to the specification/trace directory labels Nov 9, 2020
@Oberon00 Oberon00 requested review from a team as code owners November 9, 2020 18:44
@SergeyKanzhelev
Copy link
Member

SergeyKanzhelev commented Nov 9, 2020

@bogdandrutu assigning to you to coordinate both PRs.

@yurishkuro yurishkuro merged commit 06638fc into open-telemetry:master Nov 11, 2020
@Oberon00 Oberon00 deleted the array-no-null branch November 11, 2020 08:00
djc added a commit to djc/opentelemetry-rust that referenced this pull request Nov 11, 2020
djc added a commit to djc/opentelemetry-rust that referenced this pull request Nov 11, 2020
jtescher pushed a commit to open-telemetry/opentelemetry-rust that referenced this pull request Nov 11, 2020
Gmanboy added a commit to Gmanboy/opentelemetry-rust that referenced this pull request Aug 12, 2024
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:api Cross language API specification issue spec:resource Related to the specification/resource directory spec:trace Related to the specification/trace directory
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Spec is unclear on whether supporting null in arrays is required
8 participants