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
Add a credit types field #110
Conversation
8ee7406
to
699362a
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.
Thanks!
77146ff
to
4b678f9
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.
❤️ thanks for your contribution, @hawaiigal!
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.
Indeed, thank you @hawaiigal !!
This LGTM, with just one more nit. I'll also ask the remaining OSV-Schema maintainer (@rsc) to give his signoff on this.
Signed-off-by: Kailani Chu <hawaiigal@github.com>
Signed-off-by: Kailani Chu <hawaiigal@github.com>
Signed-off-by: Kailani Chu <hawaiigal@github.com>
Signed-off-by: Kailani Chu <hawaiigal@github.com>
Signed-off-by: Kailani Chu <hawaiigal@github.com>
73f5cfd
to
6aacd0e
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.
Thanks! I've also checked with @rsc and gotten his approval offline.
As discussed in #85, we'd like to propose these changes to the OSV schema to add credit types/roles that correspond to those defined by the MITRE CVE schema.
Would appreciate any feedback or proposed changes!