Validate llm_model format in ModuleLLM#89
Conversation
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #89 +/- ##
==========================================
+ Coverage 87.61% 88.35% +0.73%
==========================================
Files 19 19
Lines 1405 1425 +20
==========================================
+ Hits 1231 1259 +28
+ Misses 174 166 -8 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
LGTM. Merging. |
Fixes #88
Adds early validation for llm_model format and a unit test for this case.