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

docs: update env vars for v0.3.20 #206

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

cheahjs
Copy link
Contributor

@cheahjs cheahjs commented Sep 7, 2024

Added missing env vars for v0.3.20.

Added a banner with the last updated version - we don't version the docs, so we expect users to be on latest.

Removed the LiteLLM section entirely to avoid confusion.

Removed RAG_WEB_SEARCH_DOMAIN_FILTER_LIST because it was never configured with an environment variable.

(large chunks of this change was LLM-generated, I have attempted to verify any obvious mistakes)

@silentoplayz
Copy link
Contributor

silentoplayz commented Sep 11, 2024

#212 is my take on updating the documented environment variables by hand, which I submitted just before I realized this PR was already open. I should have checked the open PRs beforehand! The way I made my PR feel valuable rather than pointless is by adding the environment variables that were still missing directly from your PR. You had that part already done for me, which was really helpful. After reviewing both RAW files as a comparison side-by-side, I think my PR builds upon the work already done here. I didn't intend for this to happen initially, it's just how it happened due to me not checking open PRs. Again, I apologize, but I also wanted to leave my thanks as a token of appreciation.

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.

2 participants