haxetink / tink_sql

SQL embedded into Haxe
MIT License
53 stars 16 forks source link

Schema changes #44

Closed benmerckx closed 6 years ago

benmerckx commented 6 years ago

As I proposed in #43. This has some rough edges to it, but I wanted to check if this is the right approach. I cleaned up the node and php drivers a bit too, because they were becoming unwieldy :) Two things I noticed while going through the code:

benmerckx commented 6 years ago

I believe this is mergeable now :)

back2dos commented 6 years ago

I guess a few more tests wouldn't hurt, but we gotta start somewhere :)

benmerckx commented 6 years ago

Noted. I think the tests could use a few tweaks to more easily work on them, let me start there :)