Skip to content

Update Charter / Readme.md - #874

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

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

Conversation

@darcyclarke

@darcyclarke darcyclarke commented Feb 2, 2023

Copy link
Copy Markdown
Member
  • 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

@mcollina mcollina left a comment

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.

lgtm

@mhdawson

mhdawson commented Feb 3, 2023

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?

@fraxken fraxken left a comment

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.

LGTM

@sheelpriy sheelpriy left a comment

Copy link
Copy Markdown

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