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

Nicer commit / remove git add -u #3

Merged
merged 2 commits into from
Mar 21, 2018
Merged

Nicer commit / remove git add -u #3

merged 2 commits into from
Mar 21, 2018

Conversation

petulla
Copy link
Contributor

@petulla petulla commented Mar 20, 2018

@gka

I think Jeremy Ashkenas is right: The git commit message should be nicer.

Also, this removes the git add -u command.

My understanding is it's not needed after git add -A

Reference SO thread

@petulla petulla changed the title Update to git 2.0 Nicer commit / remove git add -u Mar 20, 2018
@gka
Copy link
Owner

gka commented Mar 21, 2018

Sounds good. Mergin

@gka gka merged commit fc99ca1 into gka:master Mar 21, 2018
@petulla
Copy link
Contributor Author

petulla commented Mar 21, 2018

Separately, @gka. Have you thought of just using a git alias? This package conflicts with other popular git alias setups.

For example, in the Mathias Byens dotfiles, git go switches to a new branch, creating if necessary:

go = "!f() { git checkout -b \"$1\" 2> /dev/null || git checkout \"$1\"; }; f"

You might just make git go this alias:

go = "!f() { git add -A && git commit -m \"$1\" }; git pull; git push; f"

The nice thing about using aliases is everything is organized in one file, avoiding conflicts across software. The syntax is also less verbose.

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