A Blog post about Quarkus LangChain4j and OPA based guardrails#2785
A Blog post about Quarkus LangChain4j and OPA based guardrails#2785fabiobrz wants to merge 1 commit into
Conversation
7e9affd to
9a28c16
Compare
|
for visibility: @maxandersen @cescoffier @mariofusco 🙏 |
16a88b6 to
35e4daa
Compare
|
We generally try to avoid writing in first singular person, so maybe you may want to change this especially in the first part of the article. Check some other post to better figure out this style. |
Thanks for your review @mariofusco! I'll push changes to address your comments and suggestions. |
35e4daa to
20ac0ec
Compare
20ac0ec to
88a7280
Compare
andreaTP
left a comment
There was a problem hiding this comment.
left a couple of minor nits, great job!
88a7280 to
9a0be12
Compare
9a0be12 to
9787c93
Compare
Thanks @andreaTP! I've addressed your comments 🙂 |
|
@maxandersen, @cescoffier - This is ready according to the current reviewers. |
9787c93 to
d86e969
Compare
|
Maybe in the meantime, I can update Styra to the latest version, and mention Endive instead of Chicory. Switching temporarily back to draft. |
d86e969 to
b114c3e
Compare
|
Ok, fabiobrz/quarkus-langchain4j-opa-guards#5 is merged, switching from Chicory to Endive in the sample application. I replaced the Chicory references in the blog post references with Endive, so I am removing the "Draft" status, and this is ready for further review/merge. |
b114c3e to
6aa8715
Compare
6aa8715 to
ad4e539
Compare
|
Thanks for your review @cescoffier - hopefully I addressed your comments and remarks with the changes I just pushed, but feel free to let me know what you think. P.S.: I am updating the companion applicatoin code to align with the article contents now. |
063849d to
27f406c
Compare
|
JFYI @cescoffier, I've updated the companion app repository at https://github.com/fabiobrz/quarkus-langchain4j-opa-guards |
1dce21f to
b447a7c
Compare
cescoffier
left a comment
There was a problem hiding this comment.
Please fix the spelling of LangChain4j - I still see 3 different spelling/casing.
b447a7c to
62646c8
Compare
62646c8 to
2fcb413
Compare
Hi @cescoffier, and thanks again for your extensive review. |
9dbcaf4 to
73ec077
Compare
|
@cescoffier, just a gentle reminder to let me know whether there's anything else I can do here, since I'd like to share the article with the community 🙏 CC @andreaTP |
73ec077 to
8ead755
Compare
2c0e531 to
4829a6d
Compare
4829a6d to
f704cda
Compare
|
Thanks again @cescoffier - I pushed again to address your comments and suggestions. |
SSIA, sample code at https://github.com/fabiobrz/quarkus-langchain4j-opa-guards/tree/main
CC @andreaTP