Replies: 1 comment
-
|
I'm going to pass on this change for now but I might be willing to re-evaluate after 1.0 |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
-
I was thinking to convert all of the http response
$datainto Fluent object.Why
Fluentobjects?Instead of using this:
data_get($data, 'key...') ?? '',we can use better way to fetch keys out of array:
$data->get('message.content', '')Benefits would be:
collections()I only did it for Ollama, If you @sixlive think this would be a good idea I could convert all providers into this style of code.
see code here.
Note: this refactor does not introduces any breaking changes
Beta Was this translation helpful? Give feedback.
All reactions