Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

doc: add major version note to release guide #45054

Conversation

ruyadorno
Copy link
Member

@ruyadorno ruyadorno commented Oct 18, 2022

Adding a note on how to properly edit the the src/node_version.h file
when working on the cherry-pick step of the Release Guide when working
on a new major version release.

Signed-off-by: Ruy Adorno ruyadorno@google.com

cc @BethGriggs @RafaelGSS

@nodejs-github-bot
Copy link
Collaborator

Review requested:

  • @nodejs/tsc

@nodejs-github-bot nodejs-github-bot added the doc Issues and PRs related to the documentations. label Oct 18, 2022
@targos
Copy link
Member

targos commented Oct 20, 2022

Maybe replace "lts" with "add major" in the commit message? This is not specific to lts and the message should start with a verb.

@ruyadorno
Copy link
Member Author

ruyadorno commented Oct 20, 2022

Maybe replace "lts" with "add major" in the commit message? This is not specific to lts and the message should start with a verb.

good point! i fixed that mention in the doc file but forgot about the commit msg 😁 I'll fix it!

Adding a note on how to properly edit the the `src/node_version.h` file
when working on the cherry-pick step of the Release Guide when working
on a new major version release.

Signed-off-by: Ruy Adorno <ruyadorno@google.com>
@ruyadorno ruyadorno force-pushed the add-lts-version-update-note-on-release-guide branch from 8e5d969 to ceb3604 Compare October 20, 2022 16:17
@ruyadorno ruyadorno changed the title doc: lts version update note on the release guide doc: add major version note to release guide Oct 20, 2022
anonrig pushed a commit that referenced this pull request Oct 22, 2022
Adding a note on how to properly edit the the `src/node_version.h` file
when working on the cherry-pick step of the Release Guide when working
on a new major version release.

Signed-off-by: Ruy Adorno <ruyadorno@google.com>
PR-URL: #45054
Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>
Reviewed-By: Beth Griggs <bethanyngriggs@gmail.com>
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
@anonrig
Copy link
Member

anonrig commented Oct 22, 2022

Landed in 58cfb32

@anonrig anonrig closed this Oct 22, 2022
@ruyadorno ruyadorno deleted the add-lts-version-update-note-on-release-guide branch October 24, 2022 15:34
RafaelGSS pushed a commit that referenced this pull request Nov 1, 2022
Adding a note on how to properly edit the the `src/node_version.h` file
when working on the cherry-pick step of the Release Guide when working
on a new major version release.

Signed-off-by: Ruy Adorno <ruyadorno@google.com>
PR-URL: #45054
Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>
Reviewed-By: Beth Griggs <bethanyngriggs@gmail.com>
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
@RafaelGSS RafaelGSS mentioned this pull request Nov 1, 2022
RafaelGSS pushed a commit that referenced this pull request Nov 10, 2022
Adding a note on how to properly edit the the `src/node_version.h` file
when working on the cherry-pick step of the Release Guide when working
on a new major version release.

Signed-off-by: Ruy Adorno <ruyadorno@google.com>
PR-URL: #45054
Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>
Reviewed-By: Beth Griggs <bethanyngriggs@gmail.com>
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
danielleadams pushed a commit that referenced this pull request Dec 30, 2022
Adding a note on how to properly edit the the `src/node_version.h` file
when working on the cherry-pick step of the Release Guide when working
on a new major version release.

Signed-off-by: Ruy Adorno <ruyadorno@google.com>
PR-URL: #45054
Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>
Reviewed-By: Beth Griggs <bethanyngriggs@gmail.com>
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
danielleadams pushed a commit that referenced this pull request Dec 30, 2022
Adding a note on how to properly edit the the `src/node_version.h` file
when working on the cherry-pick step of the Release Guide when working
on a new major version release.

Signed-off-by: Ruy Adorno <ruyadorno@google.com>
PR-URL: #45054
Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>
Reviewed-By: Beth Griggs <bethanyngriggs@gmail.com>
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
danielleadams pushed a commit that referenced this pull request Jan 3, 2023
Adding a note on how to properly edit the the `src/node_version.h` file
when working on the cherry-pick step of the Release Guide when working
on a new major version release.

Signed-off-by: Ruy Adorno <ruyadorno@google.com>
PR-URL: #45054
Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>
Reviewed-By: Beth Griggs <bethanyngriggs@gmail.com>
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
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.

7 participants