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

jsonld: Add proposed "lite" schema #31

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

JPEWdev
Copy link
Contributor

@JPEWdev JPEWdev commented Mar 19, 2024

Adds a proposed method for implementing a "lite" JSON-LD SPDX format. Instead of re-specifying the spec, the lite profile is instead a small hand-curated JSON schema that simply forces the root object of the document to be a SpdxDocument. This works fine because the SpdxDocument is already supposed to encompass the document elements in its elements array, so inlining them makes sense anyway.

This means that the "lite" document is also completely valid JSON-LD, which means it can be validated by the SHACL model, or even the full JSON-LD schema.

It also means it automatically keeps up with the main schema document when it changes

Adds a proposed method for implementing a "lite" JSON-LD SPDX format.
Instead of re-specifying the spec, the lite profile is instead a small
hand-curated JSON schema that simply forces the root object of the
document to be a SpdxDocument. This works fine because the SpdxDocument
is already supposed to encompass the document elements in its elements
array, so inlining them makes sense anyway.

This means that the "lite" document is also completely valid JSON-LD,
which means it can be validated by the SHACL model, or even the full
JSON-LD schema.

It also means it automatically keeps up with the main schema document
when it changes
@JPEWdev
Copy link
Contributor Author

JPEWdev commented Mar 19, 2024

@goneall @NorioKobota

@NorioKobota
Copy link
Contributor

NorioKobota commented Mar 20, 2024

Thanks @JPEWdev !
Let me check and comment if I have any questions or want to add something.
Please give me some time.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants