Commit c12ce89
tools: abort CQ session when landing several commits
Most PRs are meant to be squashed in one commit when landing. If the
collaborator hasn't been using `fixup!` commits, the CQ lands the PR
as several commits. This change makes the CQ abort by default
when attempting to land several commits, unless there's another
label added to the PR to force squashing or landing as several commits.
Fixes: #40436
Refs: nodejs/node-core-utils#572
PR-URL: #40577
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>1 parent dd08e53 commit c12ce89Copy full SHA for c12ce89
File tree
Expand file treeCollapse file tree
2 files changed
+16
-2
lines changedOpen diff view settings
Filter options
- doc/guides
- tools/actions
Expand file treeCollapse file tree
2 files changed
+16
-2
lines changedOpen diff view settings
Collapse file
+7-1Lines changed: 7 additions & 1 deletion
- Display the source diff
- Display the rich diff
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| ||
25 | 25 | |
26 | 26 | |
27 | 27 | |
28 | | - |
| 28 | + |
29 | 29 | |
30 | 30 | |
31 | 31 | |
| ||
37 | 37 | |
38 | 38 | |
39 | 39 | |
| 40 | + |
| 41 | + |
| 42 | + |
| 43 | + |
| 44 | + |
| 45 | + |
40 | 46 | |
41 | 47 | |
42 | 48 | |
|
Collapse file
tools/actions/commit-queue.sh
Copy file name to clipboardExpand all lines: tools/actions/commit-queue.sh+9-1Lines changed: 9 additions & 1 deletion
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| ||
70 | 70 | |
71 | 71 | |
72 | 72 | |
73 | | - |
| 73 | + |
| 74 | + |
| 75 | + |
| 76 | + |
| 77 | + |
| 78 | + |
| 79 | + |
| 80 | + |
| 81 | + |
74 | 82 | |
75 | 83 | |
76 | 84 | |
|
0 commit comments