- 
                Notifications
    You must be signed in to change notification settings 
- Fork 8
fix: add security team details #75
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
Conversation
Signed-off-by: Terry Howe <[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.
@TerryHowe this is really good. we should update the page to report vulnerabilities to include the new security mailing list.
Does anything related to the mailing list (ie membership etc) need to be added to this document?
Signed-off-by: Terry Howe <[email protected]>
| 
 Done. | 
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.
Content looks good, thanks for the updates
Can you look into your DCO as it is currently failing validation
Co-authored-by: Feynman Zhou <[email protected]> Signed-off-by: Terry Howe <[email protected]>
4f6a0e4    to
    ab3040f      
    Compare
  
    | fixed the DCO issue | 
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
Similar to Helm, so details on security team procedures.