deps: require sqlalchemy for ibis SQL backend

ibis-project / ibis-bigquery

BigQuery backend for Ibis

Apache License 2.0

19 stars 18 forks source link

deps: require sqlalchemy for ibis SQL backend #144

Closed tswast closed 2 years ago

cpcloud commented 2 years ago

I can help with the failing tests here. We recently removed the difference in compilation between ibis.ifelse and ibis.where. Both are now using the ops.Where operation which compiles to an if call.

tswast commented 2 years ago

Thanks for the help @cpcloud !

cpcloud commented 2 years ago

@tswast I think this PR is ready for review if you don't mind!

cpcloud commented 2 years ago

It sort of ballooned into a bit of a CI cleanup due to having to futz with poetry-dynamic-versioning.