Would it be possible to use the same (or similar) syntax for autoclosing differential revisions as autoclosing tasks? My main issue is not so much the verbosity of "Differential Revision", rather that it must be on a line by itself. I'd like to be able to just write "Closes D1" or "Merge D1" anywhere in the commit message to get the linkage and autoclose functionality.
Description
Description
Related Objects
Related Objects
- Mentioned In
- rPab17a7d4bf84: Be more lenient when accepting "Differential Revision" in the presence of…
D17121: Be more lenient when accepting "Differential Revision" in the presence of custom ad-hoc commit message fields - Mentioned Here
- T5132: Document the special syntax you can use in commit messages to cause effects
Event Timeline
Comment Actions
This request does not describe a root problem.
Please see Contributing Feature Requests for guidance on what we're looking for in a feature request. If you aren't sure what we mean by a "root problem", see Describing Root Problems.
Comment Actions
The reason we'd rather have it on the same line is because most hg clients (including Phabricator's Diffusion, but more importantly our local TortoiseHg clients) only show the first line of any commit in the pushlog. For us to get to the revision number would require an extra action. We generally try to keep our commit messages to one line for this reason. Additionally this was something that was difficult for us to figure out as it is inconsistent with the rest of the linking functionality explained in T5132.
Comment Actions
Sorry, I still don't understand your root problem.
If you want the revision in the first line, why not just put it there? This message will parse fine:
[D123] Only this line is visible Differential Revision: https://phabricator.yourinstall.com/D123
Phabricator will ignore the reference to the revision in the subject line, but you can read it with TortiseHG.
There is no "Closes D1" or "Merge D1" syntax, so this isn't inconsistent with T5132.
Comment Actions
Btw, we just ran into an issue where the Differential Revision wasn't parsed because the newline was just a LF, phabricator seems to require CRLF. So the git log shows it on its own line with the correct syntax but the description in the differential itself shows it all on one line. Another oddity...
Comment Actions
@epriestley Any idea why the issue in my previous comment would be happening? Thanks.