litellm icon indicating copy to clipboard operation
litellm copied to clipboard

Fix function_to_dict() bugs

Open Jonarod opened this issue 8 months ago • 1 comments

Title

Fix function_to_dict() formatting bugs

Relevant issues

Fixes #8539 and #9323

Pre-Submission checklist

Please complete all items before asking a LiteLLM maintainer to review your PR

  • [ ] I have Added testing in the tests/litellm/ directory, Adding at least 1 test is a hard requirement - see details
  • [ ] I have added a screenshot of my new test passing locally
  • [ ] My PR passes all unit tests on (make test-unit)[https://docs.litellm.ai/docs/extras/contributing_code]
  • [x] My PR's scope is as isolated as possible, it only solves 1 specific problem

Unfortunately no time for testing right now. Feel free to just keep this PR as illustration .

Type

🐛 Bug Fix

Changes

Adds possibility to use arrays and enums properly.

Jonarod avatar Mar 17 '25 22:03 Jonarod

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
litellm ❌ Failed (Inspect) Mar 27, 2025 1:37am

vercel[bot] avatar Mar 17 '25 22:03 vercel[bot]

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

CLAassistant avatar Apr 22 '25 22:04 CLAassistant

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.

github-actions[bot] avatar Jul 22 '25 00:07 github-actions[bot]