Add delete-squashed-branches command#924
Merged
spacewander merged 3 commits intotj:masterfrom May 27, 2021
allejo:feature/delete-squashed-branches
Merged
Add delete-squashed-branches command#924spacewander merged 3 commits intotj:masterfrom allejo:feature/delete-squashed-branches
delete-squashed-branches command#924spacewander merged 3 commits intotj:masterfrom
allejo:feature/delete-squashed-branches
Conversation
spacewander
reviewed
May 24, 2021
spacewander
approved these changes
May 25, 2021
spacewander
reviewed
May 25, 2021
| Deletes branches that have been "squashed-merged" into a specified branch | ||
|
|
||
| ```bash | ||
| $ git delete-squashed-branches main |
Collaborator
There was a problem hiding this comment.
Should also update the doc
Contributor
Author
There was a problem hiding this comment.
How does this sound to you? 6867c79
Collaborator
There was a problem hiding this comment.
Should we also need to update the output of git delete-squashed-branches main?
Would be better if we use $ (branch) git ... to indicate the branch change.
Contributor
Author
There was a problem hiding this comment.
Sure! Is this a new convention or are there other commands that do this too?
Collaborator
There was a problem hiding this comment.
No. But I think this way will be more clear.
Contributor
Author
There was a problem hiding this comment.
How does this sound to you?
```bash
$ (feature-branch) git delete-squashed-branches main
Deleted branch dependabot/bundler/kramdown-2.3.1 (was 1d3fb00).
Deleted branch dependabot/bundler/rexml-3.2.5 (was a7e4052).
$ (main) git ...
```
Contributor
Author
There was a problem hiding this comment.
Alrighty! Amended to my last commit
spacewander
approved these changes
May 27, 2021
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Uses https://github.com/not-an-aardvark/git-delete-squashed as a basis for deleting "squashed-merged" branches.
Fixes #858
I hope I'm not too far off with this PR.