Skip to content

Update litellm integration#762

Merged
TheR1D merged 1 commit into
mainfrom
local-llm-fixes
May 6, 2026
Merged

Update litellm integration#762
TheR1D merged 1 commit into
mainfrom
local-llm-fixes

Conversation

@TheR1D
Copy link
Copy Markdown
Owner

@TheR1D TheR1D commented Apr 11, 2026

  • Bump litellm from 1.42.5 to 1.83.4
  • Keep api_key in additional_kwargs for litellm completion calls
  • Remove outdated Docker + Ollama section from README
  • Minor wording improvements in README
  • Fix unicode dash in REPL help text

- Bump litellm from 1.42.5 to 1.83.4
- Keep api_key in additional_kwargs for litellm completion calls
- Remove outdated Docker + Ollama section from README
- Minor wording improvements in README
- Fix unicode dash in REPL help text
@TheR1D TheR1D requested a review from Copilot April 11, 2026 07:43
@TheR1D TheR1D self-assigned this Apr 11, 2026
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the LiteLLM integration and refreshes user-facing documentation/text to align with the new version and improve CLI/docs polish.

Changes:

  • Bumps optional litellm dependency to 1.83.4 and adjusts integration to keep api_key in request kwargs.
  • Fixes REPL help text to use an ASCII hyphen instead of a Unicode dash.
  • Removes outdated README instructions for Docker + Ollama and tweaks wording in the Ollama tip.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

File Description
sgpt/handlers/handler.py Keeps api_key in additional_kwargs for LiteLLM completion calls.
sgpt/app.py Normalizes REPL help text dash to avoid Unicode rendering issues.
README.md Updates Ollama guidance wording and removes outdated Docker+Ollama section.
pyproject.toml Pins optional litellm dependency to 1.83.4.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread README.md
@TheR1D TheR1D merged commit 17be969 into main May 6, 2026
8 checks passed
@TheR1D TheR1D deleted the local-llm-fixes branch May 6, 2026 03:22
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