-
Notifications
You must be signed in to change notification settings - Fork 140
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 Conformance chapter #1002
Update Conformance chapter #1002
Conversation
From Bob's Word doc 2024-07-22 with few edits: - typos - translate links to a preferred format - recover corrupted text in section 5.12 using the same section from v2.2.2 Markdown file Signed-off-by: Arthit Suriyawongkul <arthit@gmail.com>
- Also fix typo in Software Profile section: "Conformance to the security profile" -> "Conformance to the Software Profile" Signed-off-by: Arthit Suriyawongkul <arthit@gmail.com>
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.
LGTM
Per spdx#1010 (comment) Signed-off-by: Arthit Suriyawongkul <arthit@gmail.com>
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 pulling this together.
see: | ||
[ECMA-404](https://ecma-international.org/publications-and-standards/standards/ecma-404/) | ||
- The JSON Schema for SPDX can be found in the | ||
[SPDX specification Git repository schemas directory](https://github.com/spdx/spdx-spec/blob/master/schemas/spdx-schema.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.
This URL: https://github.com/spdx/spdx-spec/blob/master/schemas/spdx-schema.json is not exist
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.
@bact - good catch.
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.
TO Discuss: should we merge, then update? or wait for fix here.
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.
If we can find it quickly today, maybe wait.
If not, we can have another PR to fix that.
From Bob's Word doc 2024-07-22 with few edits:
Notes