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

update cardinality text for omitted optional fields #861

Open
wants to merge 1 commit into
base: support/2.3.1
Choose a base branch
from

Conversation

lumjjb
Copy link

@lumjjb lumjjb commented Aug 22, 2023

As a follow up based on conversations on the mailing list (https://lists.spdx.org/g/Spdx-tech/topic/100823660), and on today's tech call 2023-08-22, (minutes) - . This PR is to add additional text to communicate that omission of fields with optional cardinality should be treated as signaling NOASSERTION.

@lumjjb lumjjb changed the title update conformance text for omitted optional fields update cardinality text for omitted optional fields Aug 22, 2023
@kestewart kestewart added this to the 2.3 milestone Feb 5, 2024
@kestewart kestewart modified the milestones: 2.3, 2.3.1 Apr 9, 2024
@bact
Copy link
Collaborator

bact commented Jul 11, 2024

Since the proposal here has been discussed in the Tech call already.

Do we like to also have this proposed text:

Unless specified otherwise, omission of an optional field should be interpretted as signaling NOASSERTION.

in the Conformance section of post-v3.0 version, like v3.0.1, as well?

@lumjjb @goneall

@bact bact added the doc improvement Area where the project documentation needs improvement label Oct 11, 2024
@goneall
Copy link
Member

goneall commented Oct 15, 2024

Discussed on the 15 Oct 2024 tech call - there were a couple concerns with documenting this for the 3.0 spec:

  • NOASSERTION is declaring a known unknow and we may loose the semantics of this being an intentional declaration
  • NOASSERTION in 3.0 is actually for an Object property with a range of ELEMENT

Leaving this issue open for further discussion.

@bact
Copy link
Collaborator

bact commented Oct 15, 2024

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
doc improvement Area where the project documentation needs improvement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants