Closed camAtGitHub closed 1 year ago
@camAtGitHub Thank you very much for your contribution! Good stuff!
I had an issue with sed during testing, which is probably due to the differences between mac sed and linux/normal sed. When I used the multiline prompt, I got a sed: 1: ":a;N;$!ba;s/\r{0,1}\n/\ ...": unused label 'a;N;$!ba;s/\r{0,1}\n/\\n/g'
. So I used the escape function that was recently added instead of sed.
Also you are right about renaming the OPENAI key variable, but we should do this also in the install script to not break for new installs and preferably have it in a separate .conf or .key file to isolate from the bash profile and secure it further. So this can be an issue of its own in the future.
Thank you for your work! Very neat! 🙏
Multiple commits each their own change