search

spacedentist / spr

Submit pull requests for individual, amendable, rebaseable commits to GitHub
https://getcord.github.io/spr/
MIT License
376 stars 33 forks source link

`spr land` keeps triggering the CI and block merging #154

Open lazywei opened 1 year ago

lazywei commented 1 year ago

Hi,

For some reason when I run spr land on an accepted and CI passed PR, it would re-trigger the CI and would tell me that 

  ❌  GitHub Pull Request merge failed
  🛑  GitHub: 11 of 11 required status checks have not succeeded: 10 expected and 1 pending.

And after I wait for that 11 checks finish and run spr land again, it would happen again ... as a result i wouldn't be able to merge the PR with the tool. Is this a known issue?

One possibility is I did re-arrange (via rebase) the stack after run spr diff on some PRs. Is that what messing up with spr?

Thanks

spacedentist commented 1 year ago

Sometimes spr land needs to push a commit to your PR branch just before it requests the merge. This is in the context of stacked PRs. If you do spr diff - then wait for your CI to finish - and then do spr land, does that work?