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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

doc Issues and PRs related to the documentations.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants