search

Sentdex / TermGPT

Giving LLMs like GPT-4 the ability to plan and execute terminal commands
MIT License
407 stars 95 forks source link

Modify TermGPT to use .env file instead of .txt file. #2

Closed mrteey closed 1 year ago

Sentdex commented 1 year ago

Hmm, needs more work on the .env part I think. Might be willing to take this better approach with:

from dotenv import load_dotenv load_dotenv()

In the main file, but I also like the ease of just using the text file heh.

If you can re-submit as 2 separate PRs as well, that'd be ideal.

  1. requirements file + gitignore (I can just accept this one)
  2. the .env suggestion. (I want to think more on how I really want this done).

I am not entirely certain how much longer I will be using GPT-4 or anything needing API keys, so I am not positive I want this change, but also I don't think it's really working with this PR.

Sentdex commented 1 year ago

Actually, a different PR did the .env the way I wanted and came with a requirements.txt and gitignore as well, so I've merged that one.