Abraxas-365 / langchain-rust

🦜️🔗LangChain for Rust, the easiest way to write LLM-based programs in Rust
MIT License
491 stars 63 forks source link

Fix some warnings and style #170

Closed fgsch closed 2 months ago

fgsch commented 2 months ago

Some from clippy, some flagged by my editor and some found looking at the code.

This includes a backward-breaking change (DummyMemory was misspelt). I'm not sure how you want to handle this.

prabirshrestha commented 2 months ago

I'm ok with the breaking change. I'm also ok adding a CI checks to fails builds if clippy/format fails. Ok with it to be in a different PR

prabirshrestha commented 2 months ago

Merged. Thanks!