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

Installation instructions should account for #1

Open
VersusFacit opened this issue Feb 8, 2022 · 1 comment
Open

Installation instructions should account for #1

VersusFacit opened this issue Feb 8, 2022 · 1 comment

Comments

@VersusFacit
Copy link

Small thing. Your installation instructions have a cat command.

https://github.com/Infused-Insight/fzf-dbt#install-fzf-dbt

However, when pasting into a .zshrc terminal as directed, globbing and parameter expansion forces variables to be evaluated before the command is run. Unless you do a manual paste into your .zshrc, you end up with empty variables and hence syntax errors. May want to just make it into a plain old "paste this" instruction or use a fancy shopt command or two with your here doc....or put it in a script with escapes.

@roshan-abady
Copy link

I've faced the same issue.

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

No branches or pull requests

2 participants