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

Unified Diff: git_map_branches.py

Issue 2138703003: [git-map-branches] Show CL status verbatim. (Closed) Base URL: https://chromium.googlesource.com/chromium/tools/depot_tools.git@master
Patch Set: Created 4 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: git_map_branches.py
diff --git a/git_map_branches.py b/git_map_branches.py
index f5951699d7fba0ef8e24bad08ff3984522571fe0..c00dcb95e62c0d11347ce9938cdf8444a1230872 100755
--- a/git_map_branches.py
+++ b/git_map_branches.py
@@ -140,7 +140,8 @@ class BranchMapper(object):
# retrieved.
for cl, status in status_info:
self.__status_info[cl.GetBranch()] = (cl.GetIssueURL(),
- color_for_status(status))
+ color_for_status(status),
+ status)
roots = set()
@@ -260,9 +261,12 @@ class BranchMapper(object):
# The Rietveld issue associated with the branch.
if self.verbosity >= 2:
- (url, color) = ('', '') if self.__is_invalid_parent(branch) \
- else self.__status_info[branch]
- line.append(url or '', color=color)
+ (url, color, status) = ('', '', '') if self.__is_invalid_parent(branch) \
+ else self.__status_info[branch]
+ if self.verbosity > 2:
+ line.append('{} ({})'.format(url, status) if url else '', color=color)
+ else:
+ line.append(url or '', color=color)
# The subject of the most recent commit on the branch.
if self.show_subject:
« 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