Skip to content

Add note about Node.js Repo being forked#8831

Closed
SIDDHNATH99 wants to merge 1 commit intonodejs:mainfrom
SIDDHNATH99:main
Closed

Add note about Node.js Repo being forked#8831
SIDDHNATH99 wants to merge 1 commit intonodejs:mainfrom
SIDDHNATH99:main

Conversation

@SIDDHNATH99
Copy link
Copy Markdown

Added a note indicating that the repository is a fork.

Description

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.

Added a note indicating that the repository is a fork.

Signed-off-by: Siddhnath Gharge <69086430+SIDDHNATH99@users.noreply.github.com>
@SIDDHNATH99 SIDDHNATH99 requested a review from a team as a code owner April 21, 2026 12:00
@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 21, 2026

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

Project Deployment Actions Updated (UTC)
nodejs-org Ready Ready Preview Apr 21, 2026 0:01am

Request Review

@cursor
Copy link
Copy Markdown

cursor Bot commented Apr 21, 2026

PR Summary

Low Risk
Documentation-only change with no impact on runtime behavior, APIs, or data handling.

Overview
Adds a short note to README.md stating that this repository is a fork (inserted above the “What is this repo?” section).

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

@github-actions
Copy link
Copy Markdown
Contributor

👋 Codeowner Review Request

The following codeowners have been identified for the changed files:

Team reviewers: @nodejs/nodejs-website

Please review the changes when you have a chance. Thank you! 🙏

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 6eed9e0. Configure here.

Comment thread README.md
<br />
</p>

<p>Node.js Repo forked</p>
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Personal fork note added to official project README

Medium Severity

A personal note <p>Node.js Repo forked</p> has been added to the official project README.md. This is a personal annotation relevant only to the contributor's fork and does not belong in the upstream repository's documentation. It disrupts the professional presentation of the README and provides no useful information to other contributors or users.

Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit 6eed9e0. Configure here.

@ovflowd ovflowd closed this Apr 21, 2026
@ovflowd
Copy link
Copy Markdown
Member

ovflowd commented Apr 21, 2026

Please do not post spammy PRs.

@nodejs nodejs locked and limited conversation to collaborators Apr 21, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants