oh! if we automated adding the github PR number in the commit message as a part smart-apply-patch then you could just push directly to the repo after your amend without going back to the PR.
On Thu, May 30, 2019 at 5:55 PM Andrew Purtell <[email protected]> wrote: > > Yes, the problem is after your PR is reviewed, you have to add > Signed-off-by lines in the commit message on the PR branch before merge. I > would do this like I modify local commits. git commit --amend, edit the > commit message, then push. For a PR branch you have to force push to the > review branch after editing the commit message to manually add > Signed-off-by. I forget to do that, mainly because for me the PR workflow > is new. > > On Thu, May 30, 2019 at 3:51 PM Sean Busbey <[email protected]> wrote: > > > These will only add your own, to be clear. > > > > also git am will take a --signoff option directly instead of needing an > > amend. > > > > I've brainstormed expanding smart-apply-patch to automate adding > > sign-offs from reviews. I don't think it ever reached the level of a > > jira, but if enough hbase committers find the idea useful I can put > > some work into it. > > > > On Thu, May 30, 2019 at 5:46 PM Sean Busbey <[email protected]> wrote: > > > > > > via plain git commands, presuming your local author metadata is correct: > > > > > > git commit --amend --signoff > > > > > > Or via apache yetus smart-apply-patch when pulling down the PR, again > > > presuming your local author metadata is correct. > > > > > > any one of: > > > > > > smart-apply-patch --project=hbase --committer HBASE-22461 > > > smart-apply-patch --project=hbase --plugins=github --committer 274 > > > smart-apply-patch --project=hbase --committer > > > https://github.com/apache/hbase/pull/274 > > > > > > On Thu, May 30, 2019 at 5:12 PM Andrew Purtell <[email protected]> > > wrote: > > > > > > > > The last couple of changes I've contributed by the GitHub workflow and > > both > > > > times I have forgotten to manually add Signed-off-by attribution in > > the PR > > > > before squash merging it. Will try to do better. > > > > > > > > By any chance, does anyone know of a way to add such things by the > > command > > > > line to a PR? Just wondering, because my (muscle) memory is driven by > > > > command line syntax. > > > > > > > > -- > > > > Best regards, > > > > Andrew > > > > > > > > Words like orphans lost among the crosstalk, meaning torn from truth's > > > > decrepit hands > > > > - A23, Crosstalk > > > > > -- > Best regards, > Andrew > > Words like orphans lost among the crosstalk, meaning torn from truth's > decrepit hands > - A23, Crosstalk
