search

quarkiverse / quarkus-langchain4j

Quarkus Langchain4j extension
https://docs.quarkiverse.io/quarkus-langchain4j/dev/index.html
Apache License 2.0
147 stars 88 forks source link

Update to LangChain4j 0.36 #1075

Open geoand opened 3 days ago

geoand commented 3 days ago

cc @jmartisk @edeandrea

edeandrea commented 2 days ago

So by cc'ing us, is that more for "FYI", or are you trying to cattle-prod one of us to do the work? :D

jmartisk commented 2 days ago

While we're at this, I'm willing to help with it (hopefully I'm not catching myself into a big trap here). Not sure if you have something in progress already

edeandrea commented 2 days ago

Why can't dependabot do this for us?

I understand that sometimes an upgrade causes failures which require manual intervention, but dependabot would catch that too (i.e. tests on the dependabot PR would fail and someone would have to fix the PR before merging it).

But this would catch the easy path when there weren't any failures.

jmartisk commented 2 days ago

We've disabled Dependabot for the dev.langchain4j groupId as it kinda never turns out to work without additional changes

geoand commented 2 days ago

So by cc'ing us, is that more for "FYI", or are you trying to cattle-prod one of us to do the work? :D

Trying? Demanding is more like it 😆

it kinda never turns out to work without additional changes

I think only once in all the updates we've done did we have to make 0️⃣ changes

edeandrea commented 2 days ago

So by cc'ing us, is that more for "FYI", or are you trying to cattle-prod one of us to do the work? :D

Trying? Demanding is more like it 😆

You forget, I don't work in engineering :D

it kinda never turns out to work without additional changes

I think it only worked once in all the updates we've done

Good to know!

geoand commented 2 days ago

You forget, I don't work in engineering :D

Do you want to keep it that way? 😆 😆 😆