Ted Pavlic <ted@xxxxxxxxxxxxx> wrote: > > Third in a series of patches that make bash completions more robust to > different interactive shell configurations and editors. > > [PATCH 3/3] Adds a #!bash to the top of bash completions so that editors > can recognize > it as a bash script. Also adds a few simple comments above commands that > take arguments. The comments are meant to remind editors of potential > problems that can occur when the script is sourced on systems with "set > -u." Aside from the message format... OK. The message really should have looked like this from an mbox point of view: From: Ted Pavlic <ted@xxxxxxxxxxxxx> To: git <git@xxxxxxxxxxxxxxx>, Junio C Hamano <gitster@xxxxxxxxx> Cc: "Shawn O. Pearce" <spearce@xxxxxxxxxxx> Bcc: Subject: [PATCH 3/3] bash-completion: Add internal function documentation Slightly document the internal functions of the bash completion package, so callers are more easily able to determine the expected arguments. Signed-off-by: Ted Pavlic <ted@xxxxxxxxxxxxx> --- Third in a series to improve the bash completion package, so it sucks less. contrib/completion/git-completion.bash | 15 +++++++++++++++ 1 files changed, 15 insertions(+), 0 deletions(-) diff --git a/contrib/completion/git-completion.bash See how the stuff that doesn't matter to the commit message itself goes after the "---" line? And how the subject is a niceshort, one line summary of the module impacted and the change? These show up in gitk and git shortlog, and thus in the "What's changed in git.git" newsletters Junio publishes. Its important that the subject be really short and sweet. You can put more detail above the "---" line, and it will be included in the commit when Junio applies it. This is all based on the formatting at the time of commit. Anything up to the first "\n\n" in a commit message goes into the email subject line. The rest goes into the email body, but above the "---" line. You can then edit the buffer before sending to insert non-commit message text after the "---" and before the diff stat. You can include my Ack'd by line below your Signed-off-by when you resend it. Acked-by: Shawn O. Pearce <spearce@xxxxxxxxxxx> > --- > contrib/completion/git-completion.bash | 15 +++++++++++++++ > 1 files changed, 15 insertions(+), 0 deletions(-) -- Shawn. -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
