Skip to content

Conversation

@chiehw
Copy link
Contributor

@chiehw chiehw commented Sep 26, 2025

OpenAI does not officially support this parameter, but litellm does - https://github.com/BerriAI/litellm/blob/625ed3f8cfd6288df7661682e110ed0a9ffb856b/litellm/types/llms/openai.py#L593

Pull Request Checklist

Thank you for your contribution! Before submitting your PR, please make sure you have completed the following checks:

Compatibility Check

  • I have checked whether this change affects the backward compatibility of the plugin declared in README.md
  • I have checked whether this change affects the forward compatibility of the plugin declared in README.md
  • If this change introduces a breaking change, I have discussed it with the project maintainer and specified the release version in the README.md
  • I have described the compatibility impact and the corresponding version number in the PR description
  • I have checked whether the plugin version is updated in the README.md

Available Checks

  • Code has passed local tests
  • Relevant documentation has been updated (if necessary)

@Mairuis Mairuis closed this Nov 24, 2025
@Mairuis
Copy link
Collaborator

Mairuis commented Dec 8, 2025

Please accept my sincere apologies for the incorrect closure of your pull request due to a technical issue in the trigger test. This was an operational mistake on our side, and I deeply regret any inconvenience this may have caused. (T.T)
Your PR has been reopened accordingly. Thank you for your understanding. @chiehw

@Mairuis Mairuis reopened this Dec 8, 2025
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