This appears to have stopped working on our org repos

SvanBoxel / delete-merged-branch

No more manually deleting merged branches, this lovely app does it for you.

https://github.com/apps/delete-merged-branch/

ISC License

322 stars 55 forks source link

This appears to have stopped working on our org repos #198

Closed kevincowan closed 3 years ago

kevincowan commented 3 years ago

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 3 years ago

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.

kevincowan commented 3 years ago

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

ted-prnd commented 3 years ago

same issue

SvanBoxel commented 3 years ago

Should be fixed now. Could you confirm?

SvanBoxel commented 3 years ago

Closing as fixed. Please reopen if the issue persists.

kevincowan commented 3 years ago

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

drmaciej commented 3 years ago

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

mogopz commented 3 years ago

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

dmitrii-clarifai commented 3 years ago

Same. Tried reinstall, didn't helped.

danielbachhuber commented 3 years ago

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

jbrechtel commented 3 years ago

Also not working for ours.

SvanBoxel commented 3 years ago

I'll start looking into this again tonight.

SvanBoxel commented 3 years ago

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

danielbachhuber commented 3 years ago

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

SvanBoxel commented 3 years ago

@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 3 years ago

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

drmaciej commented 3 years ago

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

SvanBoxel commented 3 years ago

@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.

drmaciej commented 3 years ago

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

rjmcneill commented 3 years ago

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

kevincowan commented 3 years ago

Working for us too! Many thanks!