spr icon indicating copy to clipboard operation
spr copied to clipboard

Every time I update I get title/message differences

Open alper opened this issue 2 years ago • 3 comments

"The Pull Request's title/message differ from the local commit's message."

I'm not sure why I'm getting this and the interface how to resolve this is also not very clear. I'm passing the --update-message flag but that still asks me to provide a message (why?).

alper avatar Aug 19 '22 13:08 alper

OK. Reading the source code, I figured it out.

I had to pass a command like: spr diff --message Rebase --update-message --all

If you pass --update-message, you also need to pass a --message which will then show up in the PR as an intermediate 'change' to show what you did.

Could we document this? Also if after a rebase I want to just update my PRs, could it be possible to do this doing just spreading diff --all?

alper avatar Aug 19 '22 15:08 alper

You should only get prompted for a message if there are actual changes in your local commit. (That could also be that you rebased your commit in the meantime.)

If the commit contents didn't change at all and you didn't rebase, then spr diff --update-message shouldn't prompt for a message.

I am aware of a problem that even after using --update-message, the next run of spr diff will show the "messages differ" warning again. I think it's just a whitespace difference (maybe GitHub adds or removes a newline at the end or something). I'll have to fix that...

sven-of-cord avatar Sep 14 '22 10:09 sven-of-cord

Any update w/ this? Currently been trying SPR out this week and this has been one of my two major pain points. Seems that even if I run the following I'll still get a title/message diff warning.

$ spr diff --all (gives warning for all current outstanding diffs)
$ spr amend --all (force local to match what GitHub has)
$ spr diff --all (gives warning still for all current outstanding diffs)

maverick1872 avatar Oct 04 '23 18:10 maverick1872