Matt Triff comments

Results 37 comments of


                                            Matt Triff

Fix custom properties for browsers

I believe this was resolved by #1001 and should be fixed in the next release.

[BUGFIX] Correct evaluation of custom properties

Thank you for the contribution @brosua!

Convert to class-based declarations, add browser-based option

@antoinevastel Anything holding this back from being merged?

Failing to install using command "npm install msnodesqlv8"

It looks like there isn't a pre-built binary for your version of Node. Since that means your system needs to build the driver from source, make sure you have installed...

The specified module could not be found.

That module should be downloaded during `npm install`. Are you running `npm install` during in your build script? If so, can you provide the output?

Can I do a PR to remove prebuild-install as dependency

What would be the benefit of switching to custom scripts? Presumably if you care about space you're using a bundler which would not include `prebuild-install` and it's dependencies in your...

fix/multiple classnames

Thanks for this! Can you merge the latest version of `master` into this branch? After that I will review it.

Bug fix for assignment of customProperties in the option template

Thanks for this! Can you add a unit test for this that would have failed previously but now passes with your change?

How to achieve sql column name auto completion?

That's right, there's a lot of database structure information that modelDBA uses to better provide suggestions. That is outside of the scope of this package. The best you can do...

Support Spark & Presto Sql

I would love to add Spark and Presto to this package, but I won't have time to do so over the next while. If you'd like to submit a pull...