Skip to content

chore(gov): remove web-standards #20

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

avivkeller
Copy link
Member

Closes #10.

Before merging this, the following steps are required:

  • @nodejs/web-standards removed as a child of @nodejs/web
  • @nodejs/web's description be set to "Responsible for the development and maintenance of Node.js's Web Presence", or similar.

@avivkeller avivkeller requested a review from a team as a code owner July 13, 2025 23:48
@avivkeller
Copy link
Member Author

CC @nodejs/TSC

@ovflowd
Copy link
Member

ovflowd commented Jul 13, 2025

The issue didn't even pass and you're opening a PR already? 🫠

@avivkeller
Copy link
Member Author

avivkeller commented Jul 13, 2025

The issue received 5 approvals and no objections after 72 hours, which I believe qualifies as a pass.

I believe this aligns with the typical process for administrative changes as outlined in the nodejs/admin repo

@ovflowd
Copy link
Member

ovflowd commented Jul 13, 2025

I'd consider it passing once it gets closed no?

@avivkeller
Copy link
Member Author

No? I would assume that the issue gets closed when the action gets taken to resolve it.

i.e. a Node.js Core Collaborator nomination issue is closed by the PR to add the nominee, not when the nomination passes.

@avivkeller
Copy link
Member Author

Bump @nodejs/TSC

@ljharb

This comment was marked as outdated.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Remove @nodejs/web-standards from @nodejs/web
5 participants