Mattias Sjödin
Mattias Sjödin
Thanks for the feedback, Matt! ### poetry You can still use `requirements.txt` with poetry by simply using the `poetry export` command. However, why `poetry` is starting to become a standard...
Absolutely. I will separate the features (poetry and pre-commit) into separate PRs. Starting with Poetry. Do you want me to cover #217 for the poetry PR as well?
> Overall changes look good! > > > > But (yes, sorry 😅) I think I noticed somewhere that duckdb versions won't be backwards compatible before v1. So it probably...
> Are these changes relevant for current version as well? Hmm not sure. I _think_ so but I can double check whenever I've some time.
> Hey, @MattiasMTS > > It has come to my attenrion that version 0.10 of duckdb started [backwards compatibility](https://duckdb.org/2024/02/13/announcing-duckdb-0100.html). > > So if you can update to the latest driver...
I think it make more sense to merge this after merging #89. Mainly because I've introduced some core changes in the latter for the Structure func. Let me know whatya...
Ah sorry I totally missed this. Yea I agree with @kndndrj that the #80 commit will be removed. I've left some TODOs in the description. Give me some time and...
Should be ready for review now @kndndrj. I refactored some of the core stuff, e.g. `getPGStructure` since we are re-using this more than just the postgres adapter at this point....
@kndndrj any objections on getting this in to unblock #77 (if it is still relevant)
> @MattiasMTS do you use this branch on your side as this project doesn't seem actively maintained? I haven't been working on this branch in particular but it should be...