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

CP-1164 Update readme with zsh instructions for completions. #109

Merged

Conversation

georgelesica-wf
Copy link
Contributor

Problem

Completion only works on Bash but some people like Zsh!

Solution

Add instructions for making the completions work with Zsh.

FYI

@evanweible-wf - you might want to test this to be sure it works in real life, I tested it on my local Zsh from Homebrew, but since I don't use Zsh normally I don't actually have a .zshrc and such and I just punched everything in at a command line.

@codecov-io
Copy link

Current coverage is 45.12%

Merging #109 into master will not affect coverage as of 3b69fa1

Powered by Codecov. Updated on successful CI builds.

@evanweible-wf
Copy link
Contributor

@georgelesica-wf I tried this out and it worked for me! We should still look into #103 though, because you have to have a copy of dart_dev installed in order to source the completions script and that could lead to completion mismatches if the version installed in a project differs from the version sourced by your bash/zsh profile.

+1

@georgelesica-wf
Copy link
Contributor Author

@evanweible-wf agreed, that looks nice. Maybe I'll take a look at that on Monday. In the meantime, I just didn't want all you Zsh users to be left out of all the fun ;-)

@trentgrover-wf
Copy link
Contributor

+1
@jayudey-wf ready for merge (docs only change)

@jayudey-wf jayudey-wf changed the title Update readme with zsh instructions for completions. CP-1164 Update readme with zsh instructions for completions. Dec 8, 2015
@jayudey-wf
Copy link
Contributor

QA +1 doc change only (evan tested the instructions as well)

Merging into master

jayudey-wf added a commit that referenced this pull request Dec 8, 2015
CP-1164 Update readme with zsh instructions for completions.
@jayudey-wf jayudey-wf merged commit 2fe5772 into Workiva:master Dec 8, 2015
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.

5 participants