Feature #875: Delete referring branches
This Pull Request fixes/closes #875.
It changes the following:
- After deleting a branch, check if there are any upstream / tracking branches and ask if user wants to delete them as well.
I followed the checklist:
- [x] I added unittests
- [x] I ran
make checkwithout errors - [x] I tested the overall application
- [x] I added an appropriate item to the changelog
Hello @extrawurst, thanks for the quick review and suggestions. I've just updated the PR with the following:
- Reverted the refactoring and added the annotation to disable clippy::too_many_lines error
- Added some assertions for
get_branch_trackersin the existing unittests - Fixed the string passed to
Action::DeleteBranch. I was passing the branch name instead of a full ref 🤦♂️
I've made the changes requested.
Please have a look at my reply about Action::DeleteRemoteBranch.
marking as Draft until the remaining comments are addressed
@alessandroasm are you still planning to move this forward?
This issue has been automatically marked as stale because it has not had any activity half a year. It will be closed in 14 days if no further activity occurs. Thank you for your contributions.