D515: phabricator: standardize colors
quark (Jun Wu)
phabricator at mercurial-scm.org
Fri Aug 25 01:15:29 UTC 2017
quark created this revision.
Herald added a subscriber: mercurial-devel.
Herald added a reviewer: hg-reviewers.
REVISION SUMMARY
Previously, the `--confirm` text could have colors but the main `phabsend`
does not. This patch adjusts the main command so it also has colors.
A default color table was added so the colors are visible by default.
REPOSITORY
rHG Mercurial
REVISION DETAIL
https://phab.mercurial-scm.org/D515
AFFECTED FILES
contrib/phabricator.py
CHANGE DETAILS
diff --git a/contrib/phabricator.py b/contrib/phabricator.py
--- a/contrib/phabricator.py
+++ b/contrib/phabricator.py
@@ -59,6 +59,15 @@
cmdtable = {}
command = registrar.command(cmdtable)
+colortable = {
+ 'phabricator.action.created': 'green',
+ 'phabricator.action.skipped': 'magenta',
+ 'phabricator.action.updated': 'magenta',
+ 'phabricator.desc': '',
+ 'phabricator.drev': 'bold',
+ 'phabricator.node': '',
+}
+
def urlencodenested(params):
"""like urlencode, but works with nested parameters.
@@ -413,9 +422,9 @@
diffmap[ctx.node()] = diff
newrevid = int(revision[r'object'][r'id'])
if revid:
- action = _('updated')
+ action = 'updated'
else:
- action = _('created')
+ action = 'created'
# Create a local tag to note the association, if commit message
# does not have it already
@@ -428,10 +437,14 @@
# Nothing changed. But still set "newrevid" so the next revision
# could depend on this one.
newrevid = revid
- action = _('skipped')
+ action = 'skipped'
- ui.write(_('D%s: %s - %s: %s\n') % (newrevid, action, ctx,
- ctx.description().split('\n')[0]))
+ drevdesc = ui.label('D%s' % newrevid, 'phabricator.drev')
+ actiondesc = ui.label(_(action), 'phabricator.action.%s' % action)
+ nodedesc = ui.label(str(ctx), 'phabricator.node')
+ desc = ui.label(ctx.description().split('\n')[0], 'phabricator.desc')
+ ui.write(_('%s - %s - %s: %s\n') % (drevdesc, actiondesc, nodedesc,
+ desc))
drevids.append(newrevid)
lastrevid = newrevid
To: quark, #hg-reviewers
Cc: mercurial-devel
More information about the Mercurial-devel
mailing list