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: Trying Wladimir's idea supporting multiline from command line
Created July 14, 2017, 3:14 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,20 @@ |
| if not opts.get('issue'): |
| # New issue, make sure title and message are set |
| + fulltitle = None |
| + |
| + if opts.get('change'): |
| + description = repo[opts['change']].description() |
| + else: |
| + description = ui.prompt('New review title: ', '') |
|
Wladimir Palant
2017/07/17 13:20:37
The variable description is being assigned but nev
f.nicolaisen
2017/07/17 13:43:41
Acknowledged.
|
| if not opts.get('title') and opts.get('change'): |
| - opts['title'] = repo[opts['change']].description() |
| - if not opts.get('title'): |
| - opts['title'] = ui.prompt('New review title: ', '') |
| + fulltitle = repo[opts['change']].description() |
| + opts['title'] = fulltitle.rstrip().split('\n')[0] |
|
Wladimir Palant
2017/07/17 13:20:37
So we are no longer asking for a title if neither
f.nicolaisen
2017/07/17 13:43:40
Acknowledged.
|
| if not opts['title'].strip(): |
| raise error.Abort('No review title given.') |
| if not opts.get('message'): |
| - opts['message'] = opts['title'] |
| + opts['message'] = opts['title'] if fulltitle is None else fulltitle |
|
Wladimir Palant
2017/07/17 13:20:37
How about:
opts['message'] = fulltitle or opts['t
f.nicolaisen
2017/07/17 13:43:40
Acknowledged.
|
| 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 »
