-
Notifications
You must be signed in to change notification settings - Fork 47
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
Remove dataLicense from CreationInfo #452
Conversation
Although I voted for this, we should wait for the resolution of spdx/change-proposal#8 before merging it. |
Agree - I'll change this to draft. Here's next steps copied from the meeting etherpad where this was discussed:
@jlovejoy @swinslow - if you could update this issue once the decision is finalized. |
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.
The changes look fine.
Resolves issue #341 Signed-off-by: Gary O'Neall <gary@sourceauditor.com>
@swinslow : any updates here, was there further discussion? |
PR #500 Payload.md mentions support for dataLicense at the physical document level. |
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.
+1 to remove dataLicense from CreationInfo, will follow up in change proposal thread regarding inclusion as an optional field in SpdxDocument
Resolves issue #341