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 a regular git commit alias. #216

Closed
wants to merge 2 commits into from
Closed

Conversation

lyda
Copy link
Contributor

@lyda lyda commented Nov 17, 2016

Git users coming from svn, hg, cvs and many other places
use "ci" as an alias to "commit." I'd note that such users
are also used to "co" being an alias for "checkout" so the
"co" alias is kind of dangerous.

Git users coming from svn, hg, cvs and many other places
use "ci" as an alias to "commit."  I'd note that such users
are also used to "co" being an alias for "checkout" so the
"co" alias is kind of dangerous.
@alerque
Copy link
Collaborator

alerque commented Nov 17, 2016

I thoroughly agree that co is a bad alias for commit. Not only is it contrary to what that command is in many other source control systems, but a log of people have git aliases for it to be checkout in git as well. It might be wise to drop that one before somebody gets burned.

@lyda
Copy link
Contributor Author

lyda commented Aug 29, 2017

I've done another version of this in PR #238 that retains the "co" alias. So pick whichever (or none) suits. I do think this one would be better though.

@alerque
Copy link
Collaborator

alerque commented Mar 29, 2021

As much as I agree with my previous self that co is a bad alias, at this point removing it is probably more disruptive than leaving it. On that topic I think something like #241 proposes with user control over all aliases is probably a better future direction that pruning existing aliases.

As for this PR, it has been obsoleted by #238 (just merged).

@alerque alerque closed this Mar 29, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants