search

newaperio / custodian

GitHub bot to manage routine tasks.
https://newaperio.github.io/custodian
MIT License
4 stars 1 forks source link

Merging a PR when another PR based on it is opened causes weirdness #14

Open lleger opened 6 years ago

lleger commented 6 years ago

Given: PR #1 is open PR #2 is open and based on PR #1

When: PR #1 is merged, Custodian deletes its branch, causing PR #2 to be closed, delabeled, and have its branch deleted.

This is a bug, since PR #2 should still remain open.

Possible fix would be to check for open PRs based on a branch before deleting it. If found, rebase those branches to master instead before deleting. I think the would solve the issue.