crane icon indicating copy to clipboard operation
crane copied to clipboard

Sqlite migrations fixed

Open cage2 opened this issue 9 years ago • 1 comments

Hello,

i was unable to use crane with sqlilte, during ~~migration~~ database creation the library complains about an "alter table" which is not supported.

The following patch try to fix that, it worked for me but i am unsure if this is a general solution (sorry for that, but i can not tests with other dbms).

Bye! C.

PS: i found the unit test failed: fixed some; i will try to address the others ASAP.

PPS: Looks like all tests passed! :)

cage2 avatar Apr 05 '16 18:04 cage2

Hello!

Any chance considering this patch for merging with the master branch? I understand that you are rewriting the whole library but I am using this in production and would be nice, in the meantime, if I could use the upstream version instead of tracking changes in my patched one. Not considering this patch could be an improvement for sqlite backend.

Any feedback is appreciated.

Bye! C.

cage2 avatar Jun 04 '16 11:06 cage2