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

Install bash completion under bash_completion.d/ #574

Closed
jbnicolai opened this issue Jul 6, 2014 · 10 comments
Closed

Install bash completion under bash_completion.d/ #574

jbnicolai opened this issue Jul 6, 2014 · 10 comments

Comments

@jbnicolai
Copy link
Contributor

If possible, it would be nice if npm install -g gulp would symlink completion/bash to /etc/bash_completion.d/ or /usr/local/etc/bash_completion.d/.

In this case, if the user has bash-completion installed and either of these directories was present the completion will 'just work', rather than requiring the step outlined in completion/README.md.

I know homebrew does the same, linking the completion to $(brew --prefix)/etc/bash_completion.d/, but there might be a better cross-platform and package manager way to do this.

@sindresorhus
Copy link
Contributor

👍 For ZSH it's /usr/share/zsh/site-functions

@jbnicolai
Copy link
Contributor Author

Cool, let me get started on a module specifically for linking completion files :)

@yocontra
Copy link
Member

yocontra commented Jul 7, 2014

cc @tkellen would you be interested in having these features in liftoff? autocompletion seems like a standard CLI thing most would want

@tkellen
Copy link

tkellen commented Jul 7, 2014

I'd totally add that if they could be installed automatically via the module @jbnicolai is thinking of building. @jbnicolai will you let me know when it is ready?

@jbnicolai
Copy link
Contributor Author

@tkellen absolutely :-)

@jbnicolai
Copy link
Contributor Author

@tkellen Up and running, called shell-completion and there's a pull request to include it in gulp over here.

Note that it's an early version, and I would appreciate any and all feedback!

jbnicolai pushed a commit to jbnicolai/gulp that referenced this issue Jul 13, 2014
@drewnoakes
Copy link

+1 for bash completion out of the box

@stringparser
Copy link

awesome :)

@havenchyk
Copy link

👍

@phated
Copy link
Member

phated commented Apr 5, 2016

Moved to gulpjs/gulp-cli#73 - Closing.

@phated phated closed this as completed Apr 5, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

8 participants