Commit f53f82cb authored by Stephen Martinis's avatar Stephen Martinis Committed by Commit Bot

`git cl split`: Fix typos in cq_dry_run code

Bug: 878117
Change-Id: Ifb8af6bfcc6467ad9b79cf6292ebb8afc8f7e949
Reviewed-on: https://chromium-review.googlesource.com/1213588Reviewed-by: 's avatarJohn Budorick <jbudorick@chromium.org>
Reviewed-by: 's avatarRyan Tseng <hinoka@chromium.org>
Commit-Queue: Stephen Martinis <martiniss@chromium.org>
parent 82849ccc
......@@ -19,6 +19,13 @@ import owners_finder
import git_common as git
# If a call to `git cl split` will generate more than this number of CLs, the
# command will prompt the user to make sure they know what they're doing. Large
# numbers of CLs generated by `git cl split` have caused infrastructure issues
# in the past.
CL_SPLIT_FORCE_LIMIT = 10
def ReadFile(file_path):
"""Returns the content of |file_path|."""
with open(file_path) as f:
......@@ -204,7 +211,7 @@ def SplitCl(description_file, comment_file, changelist, cmd_upload, dry_run,
num_cls = len(files_split_by_owners)
print('Will split current branch (' + refactor_branch + ') into ' +
str(num_cls) + ' CLs.\n')
if cq_dry_ru and num_cls > CL_SPLIT_FORCE_LIMIT:
if cq_dry_run and num_cls > CL_SPLIT_FORCE_LIMIT:
print (
'This will generate "%r" CLs. This many CLs can potentially generate'
' too much load on the build infrastructure. Please email'
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment