Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(201)

Side by Side Diff: site/dev/contrib/cqkeywords.md

Issue 1237693005: Document new CQ keyword (Closed) Base URL: https://skia.googlesource.com/skia@master
Patch Set: Created 5 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 Commit Queue Keywords 1 Commit Queue Keywords
2 ===================== 2 =====================
3 3
4 COMMIT 4 COMMIT
5 ------ 5 ------
6 6
7 If you are working on experimental code and do not want to risk accidentally 7 If you are working on experimental code and do not want to risk accidentally
8 submitting the change via the CQ, then you can mark it with "COMMIT=false". 8 submitting the change via the CQ, then you can mark it with "COMMIT=false".
9 The CQ will immediately abandon the change if it contains this option. 9 The CQ will immediately abandon the change if it contains this option.
10 To do a dry run through the CQ please use Rietveld's [dry run](https://groups.go ogle.com/a/chromium.org/forum/#!topic/chromium-dev/G5-X0_tfmok) feature. 10 To do a dry run through the CQ please use Rietveld's [dry run](https://groups.go ogle.com/a/chromium.org/forum/#!topic/chromium-dev/G5-X0_tfmok) feature.
11 11
12 COMMIT=false 12 COMMIT=false
13 13
14 The CQ will run through its list of verifiers (reviewer check, trybots, tree che ck, 14 The CQ will run through its list of verifiers (reviewer check, trybots, tree che ck,
15 presubmit check), and will close the issue instead of committing it. 15 presubmit check), and will close the issue instead of committing it.
16 16
17 NO_DEPENDENCY_CHECK=true
18
19 The CQ rejects patchsets with open dependencies. You can skip this check with th is keyword.
jcgregorio 2015/07/15 19:36:20 What is an "open dependency"?
jcgregorio 2015/07/15 20:04:51 Can we add that to the docs? On 2015/07/15 at 19:
20
17 CQ_INCLUDE_TRYBOTS 21 CQ_INCLUDE_TRYBOTS
18 22
19 Allows you to add arbitrary trybots to the CQ's list of default trybots. 23 Allows you to add arbitrary trybots to the CQ's list of default trybots.
20 The CQ will block till these tryjobs pass just like the default list of tryjobs. 24 The CQ will block till these tryjobs pass just like the default list of tryjobs.
21 25
22 This is the format of the values of this keyword: 26 This is the format of the values of this keyword:
23 27
24 CQ_INCLUDE_TRYBOTS=master1:bot1,bot2;master2:bot3,bot4 28 CQ_INCLUDE_TRYBOTS=master1:bot1,bot2;master2:bot3,bot4
25 29
26 Here are some real world examples: 30 Here are some real world examples:
(...skipping 71 matching lines...) Expand 10 before | Expand all | Expand 10 after
98 The CQ will then not run any try jobs for your change and will commit the CL as soon as the tree is open, assuming the presubmit check passes. 102 The CQ will then not run any try jobs for your change and will commit the CL as soon as the tree is open, assuming the presubmit check passes.
99 103
100 NO_MERGE_BUILDS 104 NO_MERGE_BUILDS
101 --------------- 105 ---------------
102 106
103 This keyword prevents the Skia build masters from building this commit with othe rs. Use it when your 107 This keyword prevents the Skia build masters from building this commit with othe rs. Use it when your
104 commit may have effects that you don't want mis-attributed to other commits. Jus t include the keyword 108 commit may have effects that you don't want mis-attributed to other commits. Jus t include the keyword
105 somewhere in the commit message: 109 somewhere in the commit message:
106 110
107 NO_MERGE_BUILDS 111 NO_MERGE_BUILDS
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698