SvanBoxel/delete-merged-branch

This appears to have stopped working on our org repos

Closed this issue · 21 comments

kevincowan commented

Hi - I have deleted and reinstalled the bot but it is no longer deleted merged branches. Is there anything else I should try. Does it work OK if someone has enabled the Github setting to do the same. I prefer the bot since it's easier to enable at org level. Thanks!

SvanBoxel commented

I can confirm I experienced this as well for some of my repos/orgs. I've scheduled some time later this week to debug the issue. Thanks for raising the issue.

  • 👍7
kevincowan commented

Thanks for looking into it - much appreciated. We are very grateful for your bot at ShopKeep.

ted-prnd commented

same issue

SvanBoxel commented

Should be fixed now. Could you confirm?

SvanBoxel commented

Closing as fixed. Please reopen if the issue persists.

kevincowan commented

Unfortunately this is still not working for us. Is there anything I can do to help debug it?

drmaciej commented

@SvanBoxel same in our case - branches still survive.
Should we try and reinstall?

mogopz commented

Hey @SvanBoxel, this is also not working for repos in my org

  • 👍1
dmitrii-clarifai commented

Same. Tried reinstall, didn't helped.

danielbachhuber commented

Still not working for us too. Thanks for your work on this though!

jbrechtel commented

Also not working for ours.

SvanBoxel commented

I'll start looking into this again tonight.

  • 👍3
  • ❤️8
SvanBoxel commented

Thanks for your patience folks. I just released a new version and it should be fixed now.

danielbachhuber commented

@SvanBoxel Do we need to remove the application and add it again? It doesn't seem to be working for me yet.

SvanBoxel commented

@danielbachhuber That surprises me. I've tested it here and the bot deletes the branch when it is merged. Could you perhaps send me the repo slug so I can check the debug logs?

danielbachhuber commented

@SvanBoxel Sure, the repo slug is danielbachhuber/cedar.

  • 👀2
drmaciej commented

still no luck on my end, branches still survive...
should I reinstall?
have you found anything in @danielbachhuber's logs?

SvanBoxel commented

@maciejqantas My bad, my deployment script was overwriting an env variable after x minutes causing it to fail (again). This is fixed now. Could you check again? Please share the repo slug if it is still failing.

image

drmaciej commented

@SvanBoxel that fixed it :) I just merged a PR and the branch was deleted.
Thanks a lot for your work!

  • 👍2
rjmcneill commented

@SvanBoxel working for us as well now, thanks very much for the quick fix! 🎉

  • 👍1
kevincowan commented

Working for us too! Many thanks!