Skip to content

Enable command autocompletion when local installation is used #486

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

Merged
merged 1 commit into from
May 19, 2015

Conversation

rosen-vladimirov
Copy link
Contributor

When you do not specify -g flag, we could still enable autocompletion, but currently this process fails as we are executing globally installed CLI (for example tns completion >> ...). Use the javascript from bin directory instead.
Fix dev-preuninstall command, which was failing due to incorrect service called in it (cancellationService is not called at all, so I removed it. In fact we have $cancellation, not $cancellationService).

@ns-bot
Copy link

ns-bot commented May 18, 2015

@teobugslayer
Copy link
Contributor

👍 after reasonably approximate to a green build

When you do not specify -g flag, we could still enable autocompletion, but currently this process fails as we are executing globally installed CLI (for example tns completion >> ...). Use the javascript from bin directory instead.
Fix dev-preuninstall command, which was failing due to incorrect service called in it (cancellationService is not called at all, so I removed it. In fact we have $cancellation, not $cancellationService).
@rosen-vladimirov rosen-vladimirov force-pushed the vladimirov/autocompletion-enable-local branch from f62829e to c7f8b3c Compare May 19, 2015 05:24
@ns-bot
Copy link

ns-bot commented May 19, 2015

rosen-vladimirov added a commit that referenced this pull request May 19, 2015
…nable-local

Enable command autocompletion when local installation is used
@rosen-vladimirov rosen-vladimirov merged commit 6bbae8a into master May 19, 2015
@rosen-vladimirov rosen-vladimirov deleted the vladimirov/autocompletion-enable-local branch May 19, 2015 05:35
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