On Mon, Mar 9, 2015 at 1:00 AM, James [off-list ref] wrote:
This is a patch to add a much needed option to the bash completion
script. I'm not subscribed to this list, so please include me in your
reply if you'd like me to see your response.
Please read the guide on how to submit patches [1].
Although the change is trivial, some immediate problems I see are:
* The patch is attached rather than being inlined.
* The fix should also be made for the other shells (zsh and tcsh)
* The commit message contains an unnecessary smiley, personal
digression, and a period in the first line.
* The change or the problem it is solving is not clearly described. I
think it would be better if you write something like this in the first
line:
"completion: add --recursive flag to git clone"
And for good measure, add another sentence below explaining why this
should be available for completion.
Do a `git log --grep completion` to see how others have formulated
changes in the same place in the past. You want to be consistent with
them.
[1] https://raw.githubusercontent.com/git/git/master/Documentation/SubmittingPatches