Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Any Plans or Interest in Moving to HoneySQL 2? #92

Open
cmiles74 opened this issue Mar 18, 2022 · 5 comments
Open

Any Plans or Interest in Moving to HoneySQL 2? #92

cmiles74 opened this issue Mar 18, 2022 · 5 comments

Comments

@cmiles74
Copy link

Hi @camsaul!

I'm working on a project that uses Toucan and HoneySQL 2.x. I've run into a couple bugs where people try to feed a structured HoneySQL 2 query into Toucan. It's not a huge deal, but...

I was wondering: are there any plans (or even interest) in moving to HoneySQL 2?

@skyfion
Copy link

skyfion commented Oct 10, 2022

It's a good idea,
Is it enough to override honeysql->sql function for this ?

@cmiles74
Copy link
Author

It might be as easy as swapping the functions. I was looking around a little to try and gauge how difficult it would be... The new version does try to stay compatible with the old.

Mostly I was wondering if this project is interested in moving or if they want to stay with the current version.

@skyfion
Copy link

skyfion commented Oct 10, 2022

Mostly I was wondering if this project is interested in moving or if they want to stay with the current version.

it looks like not 🙂
But migrations from hsql 1 to 2 become a bit more complicated

@flamber
Copy link

flamber commented Oct 10, 2022

Have a look here https://github.com/camsaul/toucan2#toucan2-toucan1

@vldmr-k
Copy link

vldmr-k commented Mar 14, 2023

Better choice https://github.com/HealthSamurai/dsql
It is flexible and extendable lib

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants