Skip to content

Update Charter / Readme.md#874

Merged
RafaelGSS merged 2 commits intonodejs:mainfrom
darcyclarke:patch-1
Jun 22, 2023
Merged

Update Charter / Readme.md#874
RafaelGSS merged 2 commits intonodejs:mainfrom
darcyclarke:patch-1

Conversation

@darcyclarke
Copy link
Copy Markdown
Member

@darcyclarke darcyclarke commented Feb 2, 2023

  • 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"

Notably, this does remove an item from the "Charter" but that may be reasonable/within scope.

/cc @mhdawson

- 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"
@darcyclarke darcyclarke changed the title Update Readme.md Update Charter / Readme.md Feb 2, 2023
Copy link
Copy Markdown
Member

@mcollina mcollina left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@mhdawson
Copy link
Copy Markdown
Member

mhdawson commented Feb 3, 2023

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?

Copy link
Copy Markdown
Member

@fraxken fraxken left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Copy Markdown

@sheelpriy sheelpriy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@UlisesGascon
Copy link
Copy Markdown
Member

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?

@mhdawson It was done in nodejs/TSC@07153d0

Comment thread README.md
[![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
Copy link
Copy Markdown
Member

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants