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 'calm' short for commit all with a message #40

Closed
wants to merge 1 commit into from

Conversation

akalcik
Copy link

@akalcik akalcik commented Aug 19, 2018

No description provided.

@joelparkerhenderson
Copy link
Member

Thanks! How often do you personally use this command, without any push?

Do you know about these two?

  • git c -am (this is "c" for commit, then the typical built in -am for --all --message)
  • git put (this is a git alias that does commit --all --message and also does the push)

I'm aiming to make "ca" always "commit --amend" because the "--amend" arg has no short form.

@akalcik
Copy link
Author

akalcik commented Aug 21, 2018

Well I wasn't sure about the abbreviation. I thought cma as git --message --all but it wouldn't respect the order. I was aware of git put as I also had short code for this on my mind but skipped it in this pull request because of this. I know git c -am but thought shortcut would be better. I'm using it often as I commit only small changes and than push it when everything out when it make sense. I'm open to change the name.

@felipecrs
Copy link
Contributor

I have the following:

caa = commit --amend --all
caane = commit --amend --all --no-edit

And I use git caane a lot.

@joelparkerhenderson
Copy link
Member

@felipecrs Seems reasonable to me... would you like to do a merge request aka PR, or would you prefer I add your two at the next update?

@felipecrs
Copy link
Contributor

Sure, I'll do it later today. :)

@joelparkerhenderson
Copy link
Member

Merged in #100.

This pull request was closed.
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.

3 participants