From c4906d2ad9ac848df14c88f5d098328d7e82ab43 Mon Sep 17 00:00:00 2001 From: Madhav Jivrajani Date: Thu, 13 May 2021 21:08:28 +0530 Subject: Add documentaion for tide/squash-on-merge to pull-request.md and contributor-cheatsheet - adding documentation increases awareness that this label exists and can help PRs get merged quicker - Manual squash causes removal of lgtm label and re-run of CI tests. Signed-off-by: Madhav Jivrajani --- contributors/guide/contributor-cheatsheet/README.md | 2 +- contributors/guide/pull-requests.md | 2 ++ 2 files changed, 3 insertions(+), 1 deletion(-) diff --git a/contributors/guide/contributor-cheatsheet/README.md b/contributors/guide/contributor-cheatsheet/README.md index e1722f2e..716eda4e 100644 --- a/contributors/guide/contributor-cheatsheet/README.md +++ b/contributors/guide/contributor-cheatsheet/README.md @@ -374,7 +374,7 @@ git rebase -i HEAD~3 git push --force ``` - +**Note**: you can also ask your reviewer to add the `tide/merge-method-squash` label to your PR (this can be done by a reviewer by issuing the command: `/label tide/merge-method-squash`), this will let the bot take care of squashing _all_ commits that are part of this PR and will not result in removal of the `LGTM` label (if already applied) or re-run of the CI tests. [contributor guide]: /contributors/guide/README.md [developer guide]: /contributors/devel/README.md diff --git a/contributors/guide/pull-requests.md b/contributors/guide/pull-requests.md index e556af8d..9bcc7c8f 100644 --- a/contributors/guide/pull-requests.md +++ b/contributors/guide/pull-requests.md @@ -291,6 +291,8 @@ For more information, see [squash commits](./github-workflow.md#squash-commits). Don't squash when there are independent changes layered to achieve a single goal. For instance, writing a code munger could be one commit, applying it could be another, and adding a precommit check could be a third. One could argue they should be separate pull requests, but there's really no way to test/review the munger without seeing it applied, and there needs to be a precommit check to ensure the munged output doesn't immediately get out of date. + +**Note**: you can also ask your reviewer to add the `tide/merge-method-squash` label to your PR (this can be done by a reviewer by issuing the command: `/label tide/merge-method-squash`), this will let the bot take care of squashing _all_ commits that are part of this PR and will not result in removal of the `LGTM` label (if already applied) or re-run of the CI tests. ## Commit Message Guidelines -- cgit v1.2.3