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

make creation of git aliases optional #310

Closed
wants to merge 1 commit into from
Closed

make creation of git aliases optional #310

wants to merge 1 commit into from

Conversation

kljohann
Copy link

As I do not use the git aliases provided by the git module but do not want to miss out on the defined functions I propose aliases to be optional (on by default). Is :prezto:module:gitaliases okay?

@ColinHebert
Copy link
Contributor

This isn't a bad idea, it works the same way color or case-sensitive does.

If this change is accepted, every module should have this option and zstyle ':prezto:*:*' alias 'yes' should be set in the .zpreztorc

@sorin-ionescu
Copy link
Owner

Why don't you use the Git aliases? They are awesome. I'm worried about adding too many zstyles.

@kljohann
Copy link
Author

I guess for me there are just to many aliases with names that are too similar so a typo could have annoying consequences. Also I already have aliases in ~/.gitconfig for stuff I use most of the time.

@sorin-ionescu
Copy link
Owner

So, you have not even tried to use them and just dismissed them.

I have been using them, and typo problems are minimal; they rarely occur.

They are similar for consistency. You just have to memorise the middle letter.

@nibblebot
Copy link

aw man. what a terrible reason to reject this :(
lots of people use git's builtin alias ability to not conflict with the global shell aliases.
polluting the alias space with ~100 aliases is a little aggressive as a default option if you just want git prompt functions, much less having no easy way to turn it off :(

@sorin-ionescu
Copy link
Owner

One man's pollution is another man's treasure.

They are superior to gitconfig aliases.

Had you tried to use them, I might have been more accommodating, but you dismissed them outright.

@escobera
Copy link

escobera commented Nov 7, 2012

Having a way to disable the aliases creation will not remove the value from it. I use and like them, but I can see where some other ppl can disagree with the naming convention or even not wanting that many aliases and, for those, a way to disable the aliases creation would be the real treasure.

@bwiggs
Copy link

bwiggs commented Dec 18, 2012

+1

@sorin-ionescu Even if they're great, they're getting in the way of what I'm comfortable with. I might learn to like them, but they're constantly being interpreted as misspellings or conflicting with tools im already using, ex: grc. It'd be nice to be able to turn them off, and then turn them back on later.

@rizzatti
Copy link
Contributor

@bawigga you can check pull #341 too

@varemenos
Copy link

Well, back to oh-my-zsh I guess

@sorin-ionescu
Copy link
Owner

What do you mean? Modify Prezto as you see fit. OMZ also has Git aliases.

On Jan 7, 2014, at 14:29, "Adonis K." [email protected] wrote:

Well, back to oh-my-zsh I guess


Reply to this email directly or view it on GitHub.

@jeffcox
Copy link

jeffcox commented Jan 7, 2014

Isn't the whole point of a project like Prezto to maintain your own fork?

@nasenatmer
Copy link
Contributor

I like those aliases too, though I'm constantly grepping ~/.zprezto/modules/git/alias.zsh on what each of them is doing, but a zstyle to let the user decide whether they like them or not doesn't sound like too much complication to me. Hence, 👍 and dito @escobera

@mfussenegger
Copy link
Contributor

could the modules be split in order to avoid the zstyle? git-aliases/ git-functions?

@sorin-ionescu
Copy link
Owner

No, the point of a module is to be self contained.

@pushrax
Copy link

pushrax commented May 16, 2015

@sorin-ionescu is there a particularly compelling reason not to have this?

@sorin-ionescu
Copy link
Owner

No.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.