Skip to content

doc: clarify fixes and refs trailer guidance#64421

Open
trivikr wants to merge 2 commits into
nodejs:mainfrom
trivikr:doc-commit-body-refs
Open

doc: clarify fixes and refs trailer guidance#64421
trivikr wants to merge 2 commits into
nodejs:mainfrom
trivikr:doc-commit-body-refs

Conversation

@trivikr

@trivikr trivikr commented Jul 11, 2026

Copy link
Copy Markdown
Member

The existing guidance is ambiguous: it first suggests adding Fixes: and
Refs: directly to the commit message, then says the landing tooling
automatically copies them from the pull request description.

This change clarifies that these trailers should normally be included in the
pull request description, avoiding unnecessary duplication. It also documents
the manual landing case and clarifies that the sample shows the final commit
message after landing.

Signed-off-by: Kamat, Trivikram <16024985+trivikr@users.noreply.github.com>
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Review requested:

  • @nodejs/tsc

@nodejs-github-bot nodejs-github-bot added the doc Issues and PRs related to the documentations. label Jul 11, 2026
Comment thread doc/contributing/pull-requests.md Outdated
Co-authored-by: Antoine du Hamel <duhamelantoine1995@gmail.com>
@trivikr trivikr requested a review from aduh95 July 11, 2026 17:18
@trivikr trivikr added author ready PRs that have at least one approval, no pending requests for changes, and a CI started. commit-queue-rebase Add this label to allow the Commit Queue to land a PR in several commits. commit-queue Add this label to land a pull request using GitHub Actions. commit-queue-squash Add this label to instruct the Commit Queue to squash all the PR commits into the first one. and removed commit-queue-rebase Add this label to allow the Commit Queue to land a PR in several commits. labels Jul 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author ready PRs that have at least one approval, no pending requests for changes, and a CI started. commit-queue Add this label to land a pull request using GitHub Actions. commit-queue-squash Add this label to instruct the Commit Queue to squash all the PR commits into the first one. doc Issues and PRs related to the documentations.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants