From: Jan Durovec Date: Tue, 19 Apr 2016 21:25:05 GMT Subject: Re: [PATCH v2] git-p4: add P4 jobs to git commit message Message-ID: In-Reply-To: There's a comment on PR itself (in addition to individual commits) so theoretically it could. It seems that for [PATCH ... n/m] e-mails the commit messages are used, so there's no reason why the PR comment couldn't be used for a cover letter. In this case the PR comment was the same as for one of the commits. Maybe submitGit tried to be smart there? Or maybe it just really doesn't support it (yet?). On Tue, Apr 19, 2016 at 11:09 PM, Junio C Hamano wrote: > Jan Durovec writes: > >> On Tue, Apr 19, 2016 at 8:50 PM, Jan Durovec wrote: >>>> Any submitGit users? I think it lets you throw multiple-patch >>>> series just fine. In this case, you'd prepare a two patch series on >>>> a branch, 1/2 being the clean-up and 2/2 being the new feature, and >>>> if you give that branch to submitGit as a whole it should do the >>>> right thing, I'd imagine. >>> >>> Hm... I'll see what it does with a pull request that has 2 commits. >> >> Huh... seems that it works :) >> >> v3 sent in 2 parts > > Thanks. > > For a series this small it does not matter, but anything longer it > would be easier to review with a cover letter (i.e. [PATCH 0/N]). I > do not know if submitGit lets us do that, though.