Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add refs bind to delete branch. #270

Closed
wants to merge 1 commit into from

Conversation

Projects
None yet
3 participants
@cirosantilli
Copy link
Contributor

commented Apr 17, 2014

Fixes #133.

Mnemonic: deLete, since the ideal D is already taken up, by... an option! If we implement #272, D would be free =)

@jonas

This comment has been minimized.

Copy link
Owner

commented Apr 17, 2014

What about ! instead of L? ! is harder to use by accident and is already used in the status and stage view for reverting changes. While we are at it let's also add binding to the stash view to delete stashes.

Please also change add a Fixes #133 line at the end of the commit message and an entry to the NEWS file.

@cirosantilli

This comment has been minimized.

Copy link
Contributor Author

commented Apr 17, 2014

! is OK with me.

Consider adding those guidelines:

  • add contribs to NEWS
  • not fixes #XXX on commit message

to a CONTRIBUTING.md file. I searched for it =)

Do I have to modify NEWS.html myself in addition to NEWS.adoc? If its made from make NEWS.html, do we really need to store it in the repo, considering it:

  • is an output file
  • it renders well on GitHub

This would be another good point for the CONTRIBUTING.md

@jonas

This comment has been minimized.

Copy link
Owner

commented Apr 19, 2014

Thanks, merged.

Good idea with a CONTRIBUTING file.

@jonas jonas closed this Apr 19, 2014

@Endle

This comment has been minimized.

Copy link

commented Oct 18, 2014

Will tig be able to delete remove branches?

@jonas

This comment has been minimized.

Copy link
Owner

commented Oct 18, 2014

Do you mean "delete remote branches"? You can add a binding yourself:

bind refs D ?git push %(remote) :%(ref)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.