Unified Diff: hgreview.py
Issue 29371804:
Noissue - [hgreview] Only use first line of commit message as review title (Closed)
Base URL: https://hg.adblockplus.org/codingtools/
Patch Set: Remove unnecessary condition
Created July 18, 2017, 1:48 p.m.
Use n/p to move between diff chunks;
N/P to move between comments.
« no previous file with comments
|
« no previous file
|
no next file »
|
no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
| Index: hgreview.py |
| =================================================================== |
| --- a/hgreview.py |
| +++ b/hgreview.py |
| @@ -58,15 +58,19 @@ |
| if not opts.get('issue'): |
| # New issue, make sure title and message are set |
| - if not opts.get('title') and opts.get('change'): |
| - opts['title'] = repo[opts['change']].description() |
| - if not opts.get('title'): |
| + fulltitle = None |
| + |
| + if not opts.get('title') and not opts.get('change'): |
| opts['title'] = ui.prompt('New review title: ', '') |
| + elif not opts.get('title'): |
| + fulltitle = repo[opts['change']].description() |
| + opts['title'] = fulltitle.rstrip().split('\n')[0] |
| + |
| if not opts['title'].strip(): |
| raise error.Abort('No review title given.') |
| if not opts.get('message'): |
| - opts['message'] = opts['title'] |
| + opts['message'] = fulltitle or opts['title'] |
| path = (ui.config('paths', 'default-push') |
| or ui.config('paths', 'default') |
« no previous file with comments
|
« no previous file
|
no next file »
|
no next file with comments »
