Infused-Insight / fzf-dbt

An integration for dbt and fzf that allows interactive selection and search of dbt models.
66 stars 5 forks source link

Installation instructions should account for #1

Open VersusFacit opened 2 years ago

VersusFacit commented 2 years ago

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 commented 1 year ago

I've faced the same issue.