Skip to content

Conversation

@Flamenco
Copy link

This is already implemented in the mysql driver...

This is already implement in the mysql driver...
@kylefarris
Copy link
Owner

Thanks for the PR. I'll get this patched in ASAP.

@Flamenco
Copy link
Author

It works for me, but you might want to add a test case.

When will you be releasing the next version? I will promisify it, and have been holding back on working on the current version.

@kylefarris
Copy link
Owner

I'm hoping to have v2 out on August 15th.

@kylefarris
Copy link
Owner

kylefarris commented Jul 16, 2018

@Flamenco The code is being heavily-refactored to use ES2017 class syntax. You can see the progress being made (and the new code structure) if you look at the mssql branch. Of course, MSSQL support is coming as well.

@kylefarris kylefarris added this to the v2 milestone Jul 26, 2018
@kylefarris kylefarris added the bug label Jul 26, 2018
@kylefarris kylefarris self-assigned this Jul 26, 2018
@kylefarris
Copy link
Owner

I'm going to have to manually merge this into the v2 branch since I can't pull this request into it (too many changes). Looks good--thanks for the work you put into it.

@Flamenco
Copy link
Author

Flamenco commented Aug 3, 2018

Closing this as it is fixed in v2

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants