-
Notifications
You must be signed in to change notification settings - Fork 122
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 Charter / Readme.md #874
Conversation
- remove reference to legacy nsp (Node Security Project) which was acquired by npm inc. (which was then acquired by GitHub) - remove "ecosystem" prefix for the WG as this has been promoted & is canonically known as Node.js's "Security WG"
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
Went to look at governance etc. related to changing. I think we should change here first: https://github.com/nodejs/TSC/blob/main/WORKING_GROUPS.md#ecosystem-security @darcyclarke interested/have time to submit PR there or would you like me to do it? |
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
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
@mhdawson It was done in nodejs/TSC@07153d0 |
@@ -3,7 +3,8 @@ | |||
[![Security WG Twitter Hashtag](https://img.shields.io/badge/Twitter-%23SecurityWG-blue.svg)](https://twitter.com/search?q=SecurityWG) | |||
[![OpenJS Slack Invite](https://img.shields.io/badge/join%20slack%20on-nodejs--security--wg-green.svg)](https://slack-invite.openjsf.org/) | |||
[![OpenSSF scorecard](https://api.securityscorecards.dev/projects/github.com/nodejs/security-wg/badge)](https://api.securityscorecards.dev/projects/github.com/nodejs/security-wg) | |||
# Ecosystem Security Working Group | |||
|
|||
# Security Working Group |
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 and similar places should now be updated to say team
instead of working group
.
Notably, this does remove an item from the "Charter" but that may be reasonable/within scope.
/cc @mhdawson