Skip to content

feat(ci): add codeowners#60

Open
avivkeller wants to merge 1 commit intomainfrom
codeowners
Open

feat(ci): add codeowners#60
avivkeller wants to merge 1 commit intomainfrom
codeowners

Conversation

@avivkeller
Copy link
Copy Markdown
Member

@avivkeller avivkeller commented Apr 25, 2026

Fixes #9. This is my best guess on who should own each file, but cc @nodejs/TSC to double check.

Copilot AI review requested due to automatic review settings April 25, 2026 21:20
@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 25, 2026

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

Project Deployment Actions Updated (UTC)
nodejs-learn Ready Ready Preview Apr 25, 2026 9:20pm

Request Review

@cursor
Copy link
Copy Markdown

cursor Bot commented Apr 25, 2026

PR Summary

Medium Risk
Adds a new pull_request_target GitHub Action with pull-requests: write permissions to post review-request comments, which is moderately sensitive if misconfigured. Otherwise changes are limited to review routing/automation and do not affect runtime code.

Overview
Adds a new .github/CODEOWNERS file to route reviews to specific Node.js working groups based on documentation path patterns, with repo-wide defaults and a catch-all for infra files.

Adds a Request Codeowner Reviews GitHub workflow that runs on PR open (pull_request_target) and uses pkgjs/request-codeowner-review to automatically request the relevant CODEOWNERS and leave the result as a PR comment.

Reviewed by Cursor Bugbot for commit d433831. Bugbot is set up for automated code reviews on this repo. Configure here.

Copy link
Copy Markdown

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit d433831. Configure here.

Comment thread .github/CODEOWNERS
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 repository ownership metadata and an automation workflow to request CODEOWNERS-based reviews on newly opened pull requests (Fixes #9).

Changes:

  • Introduces a CODEOWNERS file under .github/ mapping content areas to Node.js teams.
  • Adds a GitHub Actions workflow to request codeowner reviews on pull_request_target (opened) events.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
.github/workflows/request-codeowner-review.yml New workflow to request CODEOWNERS-based reviews for newly opened PRs.
.github/CODEOWNERS New CODEOWNERS rules assigning ownership for pages and dotfiles.

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

Comment thread .github/workflows/request-codeowner-review.yml
Comment thread .github/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

2 participants