Open opqpop opened 1 year ago
Hmm it might be because we've recently added pre-commit hooks that require an approval before it can be merged, as well as passing a build check
but my approval + build check is all against the parent diff instead of the main branch, so when spr land swaps it to be against the main branch, it fails the pre-commit hook until the newly build check that got kicked off finishes.
Any suggestions for how to best get around this? Is there a way for spr to bypass pre-commit hooks vs main, as long as they passed when it was against the parent diff?
we ran into a similar issue recently, ours occured in a flow like this.
This has just started happening 2-3 days ago and I'm not sure why. Is it possible it has to do with the recent commits that happened since 3 days ago? Pinging the authors too in case it might be related (https://github.com/getcord/spr/pull/159 cc @andrewhamon, https://github.com/getcord/spr/pull/153 cc @cadolphs, https://github.com/getcord/spr/pull/162 cc @spacedentist)
Repro:
git checkout origin/main; git pull origin main --rebase
git cherry-pick 123
spr diff -m "rebase"
spr land
, get below errorLet me know if there's anything else I can help with