feature: added sqlite. Fixes #19

[swampie]

This PR adds the drivers and the support for sqlite databases

[pditommaso]

This is nice. What about adding a test for the binary version like this so we are sure the compile version works?

[swampie]

This is nice. What about adding a test for the binary version like this so we are sure the compile version works?

Native test running as we speak

[pditommaso]

Task :nativeCliTest

SqliteTest > should run native binary FAILED
java.lang.ArrayStoreException at SqliteTest.groovy:33

[swampie]

I love GraalVM

[jorgeaguileraseqera]

It's not a graalvm error,

change this line
--url', "jdbc:sqlite:${this.dbName}"
by

'--url', "jdbc:sqlite:${this.dbName}".toString(),

[jorgeaguileraseqera]

also, I don't know why, you need to specify the driver ( '--driver','org.h2.Driver',) in command-line arguments

maybe we are missing to include some dependency

[pditommaso]

Actually it should be needed. sqlite should be added here

http://github.com/seqeralabs/migtool/blob/0a1d13300d62666d956e404b5ea0f1f6100fd9e8/src/main/java/io/seqera/migtool/Helper.java#L204-L218

[pditommaso]

Let's invoke @jorgeaguileraseqera superpower for this build issue

[pditommaso]

The graal guru strike again 😄

[jorgeaguileraseqera]

stackoverflow superpower

btw I founded we're initilizing the dialect to mysql always

[pditommaso]

btw I founded we're initilizing the dialect to mysql always

What do you mean?

[jorgeaguileraseqera]

sorry:

at https://github.com/seqeralabs/migtool/pull/20/files#diff-d5a5ef22be862b9bb4e634ed65a40cfda20d8302fd85b238519a9aecb2607601R32

we init the dialect by the default to mysql so if not set by command line we use mysql and never extracted from the url

[swampie]

Is this version releasable? @pditommaso

[pditommaso]

Looks so merged