Skip to content

doc: expand on squashing and rebasing to land a PR#48751

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.

Sign up for GitHub

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

Jump to bottom
Merged
nodejs-github-bot merged 4 commits into from
Jul 15, 2023
Merged

doc: expand on squashing and rebasing to land a PR #48751

nodejs-github-bot merged 4 commits into from
Jul 15, 2023

Conversation

@legendecas
Copy link
Member

@legendecaslegendecas commented Jul 13, 2023

Document the choice of squashing and rebasing when landing a pull request containing more than one subsystem commits.

Refs: #47956 (comment)

@nodejs-github-bot
Copy link
Collaborator

nodejs-github-bot commented Jul 13, 2023

Review requested:

  • @nodejs/tsc

@nodejs-github-botnodejs-github-bot added the doc Issues and PRs related to the documentations. label Jul 13, 2023
RafaelGSS
RafaelGSS reviewed Jul 13, 2023
View reviewed changes
MoLow
MoLow approved these changes Jul 13, 2023
View reviewed changes
Copy link
Member

@MoLowMoLow left a comment

Choose a reason for hiding this comment

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

LGTM. should we mention the corresponding commit-queue-rebaseAdd this label to allow the Commit Queue to land a PR in several commits. and commit-queue-squashAdd this label to instruct the Commit Queue to squash all the PR commits into the first one. labels?

aduh95
aduh95 reviewed Jul 13, 2023
View reviewed changes
aduh95
aduh95 reviewed Jul 13, 2023
View reviewed changes
aduh95
aduh95 reviewed Jul 13, 2023
View reviewed changes
@lpincalpinca added the commit-queue-squash Add this label to instruct the Commit Queue to squash all the PR commits into the first one. label Jul 14, 2023
mhdawson
mhdawson approved these changes Jul 14, 2023
View reviewed changes
Copy link
Member

@mhdawsonmhdawson left a comment

Choose a reason for hiding this comment

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

LGTM

@aduh95aduh95 added 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. labels Jul 14, 2023
@nodejs-github-botnodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Jul 15, 2023
@nodejs-github-botnodejs-github-bot merged commit 5da84a6 into nodejs:mainJul 15, 2023
@nodejs-github-bot
Copy link
Collaborator

nodejs-github-bot commented Jul 15, 2023

Landed in 5da84a6

@legendecaslegendecas deleted the docs/pr-rebase branch July 19, 2023 03:14
Ceres6 pushed a commit to Ceres6/node that referenced this pull request Aug 14, 2023
@legendecas@Ceres6
doc: expand on squashing and rebasing to land a PR
faf73fe
PR-URL: nodejs#48751 Reviewed-By: Moshe Atlow <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Rafael Gonzaga <[email protected]> Reviewed-By: Antoine du Hamel <[email protected]> Reviewed-By: Michael Dawson <[email protected]>
Ceres6 pushed a commit to Ceres6/node that referenced this pull request Aug 14, 2023
@legendecas@Ceres6
doc: expand on squashing and rebasing to land a PR
b219e5c
PR-URL: nodejs#48751 Reviewed-By: Moshe Atlow <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Rafael Gonzaga <[email protected]> Reviewed-By: Antoine du Hamel <[email protected]> Reviewed-By: Michael Dawson <[email protected]>
UlisesGascon pushed a commit to UlisesGascon/node that referenced this pull request Aug 14, 2023
@legendecas@UlisesGascon
doc: expand on squashing and rebasing to land a PR
21b8202
PR-URL: nodejs#48751 Reviewed-By: Moshe Atlow <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Rafael Gonzaga <[email protected]> Reviewed-By: Antoine du Hamel <[email protected]> Reviewed-By: Michael Dawson <[email protected]>
RafaelGSS pushed a commit that referenced this pull request Aug 15, 2023
@legendecas@RafaelGSS
doc: expand on squashing and rebasing to land a PR
ea54784
PR-URL: #48751 Reviewed-By: Moshe Atlow <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Rafael Gonzaga <[email protected]> Reviewed-By: Antoine du Hamel <[email protected]> Reviewed-By: Michael Dawson <[email protected]>
@UlisesGasconUlisesGascon mentioned this pull request Aug 15, 2023
Merged
ruyadorno pushed a commit that referenced this pull request Sep 12, 2023
@legendecas@ruyadorno
doc: expand on squashing and rebasing to land a PR
cb565ee
PR-URL: #48751 Reviewed-By: Moshe Atlow <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Rafael Gonzaga <[email protected]> Reviewed-By: Antoine du Hamel <[email protected]> Reviewed-By: Michael Dawson <[email protected]>
ruyadorno pushed a commit that referenced this pull request Sep 13, 2023
@legendecas@ruyadorno
doc: expand on squashing and rebasing to land a PR
10495ad
PR-URL: #48751 Reviewed-By: Moshe Atlow <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Rafael Gonzaga <[email protected]> Reviewed-By: Antoine du Hamel <[email protected]> Reviewed-By: Michael Dawson <[email protected]>
@ruyadornoruyadorno mentioned this pull request Sep 13, 2023
Merged
ruyadorno pushed a commit that referenced this pull request Sep 17, 2023
@legendecas@ruyadorno
doc: expand on squashing and rebasing to land a PR
d3041df
PR-URL: #48751 Reviewed-By: Moshe Atlow <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Rafael Gonzaga <[email protected]> Reviewed-By: Antoine du Hamel <[email protected]> Reviewed-By: Michael Dawson <[email protected]>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@lpincalpincalpinca approved these changes

@MoLowMoLowMoLow approved these changes

@mhdawsonmhdawsonmhdawson approved these changes

@aduh95aduh95aduh95 approved these changes

@RafaelGSSRafaelGSSRafaelGSS approved these changes

Assignees

No one assigned

Labels

author readyPRs that have at least one approval, no pending requests for changes, and a CI started.commit-queue-squashAdd this label to instruct the Commit Queue to squash all the PR commits into the first one.docIssues and PRs related to the documentations.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

7 participants

@legendecas@nodejs-github-bot@lpinca@MoLow@mhdawson@aduh95@RafaelGSS