mirror of
https://we.phorge.it/source/arcanist.git
synced 2024-11-25 08:12:40 +01:00
Treat empty update message as user abort on 'arc diff' workflow
Summary: "-m ''" will still let you do a truly empty update if you are insistent on that. Test Plan: meta Differential Revision: 209194 Reviewed By: dschleimer Reviewers: dschleimer CC: epriestley Revert Plan: OK
This commit is contained in:
parent
89a3606406
commit
c97b736a21
1 changed files with 5 additions and 0 deletions
|
@ -833,6 +833,11 @@ EOTEXT
|
||||||
->setName('differential-update-comments')
|
->setName('differential-update-comments')
|
||||||
->editInteractively();
|
->editInteractively();
|
||||||
$comments = preg_replace('/^\s*#.*$/m', '', $comments);
|
$comments = preg_replace('/^\s*#.*$/m', '', $comments);
|
||||||
|
|
||||||
|
$comments = rtrim($comments);
|
||||||
|
if (!strlen($comments)) {
|
||||||
|
throw new ArcanistUserAbortException();
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
return $comments;
|
return $comments;
|
||||||
|
|
Loading…
Reference in a new issue