Dependencies: Update to sqlalchemy-cratedb next, remove monkeypatches - #115
Dependencies: Update to sqlalchemy-cratedb next, remove monkeypatches#115amotl wants to merge 1 commit into
Conversation
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
| langchain-classic = "<2" | ||
| langchain-postgres = "==0.0.17" | ||
| sqlalchemy-cratedb = ">=0.40.1" | ||
| sqlalchemy-cratedb = { git = "https://github.com/zerotired/sqlalchemy-cratedb.git", branch = "visit_on_conflict_do_update-ng" } |
There was a problem hiding this comment.
Needs to be resolved after another bugfix release of sqlalchemy-cratedb. Thanks in advance!
sqlalchemy-cratedb = ">=0.43.1"
About
Remove custom monkeypatches after next release of the SQLAlchemy dialect.
References
PGCompiler.visit_on_conflict_do_updatesqlalchemy-cratedb#276Review
I think CI failures show up now because the PR is originating from an external organization. In this case, they can be safely ignored.
/cc @bgunebakan, @kneth