search

php-llm / llm-chain

PHP library for building LLM-based features and applications.
MIT License
12 stars 3 forks source link

refactor: `ResponseInterface` as `Chain` return value #117

Closed chr-hertel closed 1 month ago

OskarStark commented 1 month ago

I like this

OskarStark commented 1 month ago

I added some tests for your branch here:

OskarStark commented 1 month ago

I think we can add No BC break label instead of adding BC break label 🤣

chr-hertel commented 1 month ago

i'm not awake enough for this :D why is it not a BC break?

OskarStark commented 1 month ago

I mean we nearly only introduce BC breaks, so maybe its better to opt out 😅

chr-hertel commented 1 month ago

true :angel: