yomorun / yomo

🦖 Stateful Serverless Framework for Geo-distributed Edge AI Infra
https://yomo.run
Apache License 2.0
1.64k stars 128 forks source link

fix(cli): serverless ctx mock pkg move bug #827

Closed woorui closed 1 month ago

woorui commented 1 month ago

Description

  1. Move MockContext back to serverless/mock package.
  2. Change function signature from ReadLLMFunctionCall(fnCall any) error to ReadLLMFunctionCall() (*ai.FunctionCall, error), This can avoid type missing.
vercel[bot] commented 1 month ago

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

1 Ignored Deployment | Name | Status | Preview | Comments | Updated (UTC) | | :--- | :----- | :------ | :------- | :------ | | **yomo** | ⬜️ Ignored ([Inspect](https://vercel.com/yomorun/yomo/AdP3wf7jKZqWLCHFBqJHST5vHcYL)) | [Visit Preview](https://yomo-git-fix-mock-ctx-yomorun.vercel.app) | | May 29, 2024 11:31am |
codecov[bot] commented 1 month ago

Codecov Report

Attention: Patch coverage is 83.33333% with 2 lines in your changes are missing coverage. Please review.

Project coverage is 64.75%. Comparing base (5064d07) to head (fd5a7bb).

Files Patch % Lines
ai/openai.go 66.66% 0 Missing and 1 partial :warning:
pkg/bridge/ai/service.go 0.00% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #827 +/- ## ========================================== + Coverage 64.34% 64.75% +0.40% ========================================== Files 45 45 Lines 2614 2613 -1 ========================================== + Hits 1682 1692 +10 + Misses 821 808 -13 - Partials 111 113 +2 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

fanweixiao commented 1 month ago

great job