-
Notifications
You must be signed in to change notification settings - Fork 82
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
chore: add SECURITY.md #1328
base: main
Are you sure you want to change the base?
chore: add SECURITY.md #1328
Conversation
Signed-off-by: Rano | Ranadeep <[email protected]>
Just curious if this PR's progress is blocked on anything. |
Hey @seanchen1991, we are waiting to be included in the ICF security program - which is due to happen sometime this month. |
Signed-off-by: Rano | Ranadeep <[email protected]>
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.
I've changed wording in a few places and overhauled the guidelines a bit. Review and ask if you have any questions.
Co-authored-by: Greg Szabo <[email protected]> Signed-off-by: Rano | Ranadeep <[email protected]>
Signed-off-by: Rano | Ranadeep <[email protected]>
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.
I don’t have any additional comments. Greg has already gone through it thoroughly. Just as we discussed, please check with Amulet before merging, see if they're good with this SECURITY.md
or it would be better we stick to their template, especially since we’re interested in being eventually part of the bounty program.
Closes: #824
Description
Rendered SECURITY.md
PR author checklist:
unclog
.docs/
).Reviewer checklist:
Files changed
in the GitHub PR explorer.