-
Notifications
You must be signed in to change notification settings - Fork 142
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
Validate documentation #938
Validate documentation #938
Conversation
f6e67ef
to
c96c618
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changes LGTM and it does pass the CI with the additional checking.
f8cb472
to
3615a65
Compare
docs/annexes/cross-reference.md
Outdated
Markdown file, use the following command: | ||
|
||
```shell | ||
cat getting-started.md | awk 'BEGIN{flag=0} /^```json/, $0=="```" { if (/^---$/){flag++} else if ($0 !~ /^```.*/ ) print $0 > "doc-" flag ".spdx.json"}' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
cat getting-started.md | awk 'BEGIN{flag=0} /^```json/, $0=="```" { if (/^---$/){flag++} else if ($0 !~ /^```.*/ ) print $0 > "doc-" flag ".spdx.json"}' | |
cat cross-reference.md | awk 'BEGIN{flag=0} /^```json/, $0=="```" { if (/^---$/){flag++} else if ($0 !~ /^```.*/ ) print $0 > "doc-" flag ".spdx.json"}' |
I just added the functionality to load multiple documents to my tool https://github.com/maxhbr/spdx3ToGraph and this generates the following for the cross-reference.md example: |
Very cool! |
bceef54
to
c5354db
Compare
This made me realize I forgot the |
Signed-off-by: Joshua Watt <[email protected]>
Updates the security information Annex to conform to the SPDX 3.0 schema and model. In addition, the CI workflow is updated to use a standalone script to do the validation (making it easier to validate locally). When validation documentation files, the script will now detect if the documentation is a complete document or just fragements of a document. In the latter case, a wrapper is made around the JSON from the documentation to make it a valid document for validation (e.g. "@context", "@graph", and a CreationInfo). Signed-off-by: Joshua Watt <[email protected]>
Adds a markdown document with examples that explains how to refer to elements across documents Signed-off-by: Joshua Watt <[email protected]>
b7d4466
to
c4a117c
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Very nice explanation - Thanks @JPEWdev
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for adding this in Joshua!
Validates security documentation