Skip to content

Add CODEOWNER for vulnerability blog post#8967

Open
bmuenzenmeyer wants to merge 1 commit into
mainfrom
bmuenzenmeyer-patch-1
Open

Add CODEOWNER for vulnerability blog post#8967
bmuenzenmeyer wants to merge 1 commit into
mainfrom
bmuenzenmeyer-patch-1

Conversation

@bmuenzenmeyer

@bmuenzenmeyer bmuenzenmeyer commented Jun 18, 2026

Copy link
Copy Markdown
Contributor

#8966 should be covered by releasers too?

Description

The website team is seeking more ergonomic governance of changes like this. Releasers reserve the right to forgo formal review processes, this still feels appropriate for things like #8954 etc

Validation

Related Issues

Check List

  • I have read the Contributing Guidelines and made commit messages that follow the guideline.
  • I have run pnpm format to ensure the code follows the style guide.
  • I have run pnpm test to check if all tests are passing.
  • I have run pnpm build to check if the website builds without errors.
  • I've covered new added functionality with unit tests if necessary.

#8966 should be covered by releasers too?

Signed-off-by: Brian Muenzenmeyer <brian.muenzenmeyer@gmail.com>
Copilot AI review requested due to automatic review settings June 18, 2026 21:41
@bmuenzenmeyer bmuenzenmeyer requested a review from a team as a code owner June 18, 2026 21:41
@vercel

vercel Bot commented Jun 18, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
nodejs-org Ready Ready Preview Jun 18, 2026 9:42pm

Request Review

@cursor

cursor Bot commented Jun 18, 2026

Copy link
Copy Markdown

PR Summary

Low Risk
Governance-only change to GitHub CODEOWNERS; no runtime or site build behavior is affected.

Overview
Adds apps/site/pages/en/blog/vulnerability to the Node.js Release Blog Posts section of .github/CODEOWNERS, assigning @nodejs/releasers the same way as release and announcements.

Vulnerability disclosure posts under that path will now automatically request releaser review on PRs.

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

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Adds an explicit CODEOWNERS rule so changes to the vulnerability blog section request review from the Node.js releasers team, aligning it with other release-related blog categories.

Changes:

  • Add apps/site/pages/en/blog/vulnerability ownership to @nodejs/releasers in .github/CODEOWNERS.

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

@codecov

codecov Bot commented Jun 18, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 75.68%. Comparing base (e3dbfbb) to head (57fe250).
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #8967      +/-   ##
==========================================
- Coverage   75.70%   75.68%   -0.03%     
==========================================
  Files         102      102              
  Lines        8676     8676              
  Branches      321      322       +1     
==========================================
- Hits         6568     6566       -2     
- Misses       2105     2107       +2     
  Partials        3        3              

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

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.

6 participants