Skip to content

chore: add CODEOWNERS#41

Open
abhijeetnardele24-hash wants to merge 1 commit intonodejs:mainfrom
abhijeetnardele24-hash:chore/add-codeowners
Open

chore: add CODEOWNERS#41
abhijeetnardele24-hash wants to merge 1 commit intonodejs:mainfrom
abhijeetnardele24-hash:chore/add-codeowners

Conversation

@abhijeetnardele24-hash
Copy link
Copy Markdown

Description

Adds a repository-level CODEOWNERS file so content and infrastructure changes automatically request the appropriate Node.js teams for review.

Changes

  • add a default owner for repository content
  • route .github and deployment/tooling files to @nodejs/web-infra
  • keep package.json shared between website and infra owners

Fixes #9

Copilot AI review requested due to automatic review settings April 5, 2026 07:39
@vercel
Copy link
Copy Markdown

vercel bot commented Apr 5, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
nodejs-learn Canceled Canceled Apr 5, 2026 7:39am

Request Review

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds a repository-level .github/CODEOWNERS configuration so GitHub automatically requests reviews from the appropriate Node.js teams based on which files change.

Changes:

  • Add default ownership for repository content (*) to @nodejs/nodejs-website
  • Assign infra/tooling-related files (e.g., .github/, vercel.json, doc-kit.config.mjs, lockfile) to @nodejs/web-infra
  • Keep package.json co-owned by both website and infra teams

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Copy Markdown
Member

@avivkeller avivkeller left a comment

Choose a reason for hiding this comment

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

We need to add the teams that manage each page to the CODEOWNERS

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add CODEOWNERS

3 participants