There's a benefit in having commits and diffs cross referenced outside of just autoclosing the diff. Much like the Edit Maniphest Tasks button, can there be one for diffs too?
While I agree it would be nice to have some control over this in the UI here are two comments:
- This can be done via the CLI: ./bin/differential attach-commit
- It would help if you could articulate the "benefit in having commits and diffs cross referenced outside of just autoclosing the diff". See https://secure.phabricator.com/book/phabcontrib/article/describing_problems/
Sorry, I assumed it was obvious/was being lazy.
- The cross reference is shown on the commit log in the repository which is very useful when we are looking through history and trying to understand why something was done. Usually we'll find information in diff comments.
- The commit is then properly associated with the diff in the task which further assists in analysis of code changes when there are multiple commits/diffs for a single task.
I don't understand...there's no requirement per se for a commit to have a diff attached so the commits come through with no diff attached. Usually this happens in error because the use doesn't enter the diff id correctly or misspells "Differential Revision" or complete forget to add it in, etc. So the diff is there and was properly reviewed but the association hasn't been made.
The concern here is Arcanist already solves this problem, and if you really can't use arcanist, then T5000 solves this problem. Beyond that, this feels like a band-aid and not a proper fix to the issue. There should be automatic, programmatic means of doing this. Having users attach after the fact still relies on them to do this step manually, which is unlikely to be 100% a success.