Closed timgws closed 6 years ago
Hey @timgws, sorry I got tasked to work on a different feature for a while. I'll try everything out against MySQL either today or tomorrow. Are there any additional unit tests that need to be created?
@wpanec-uno I think it's pretty much all here now. There is 100% coverage, and everything seems to work fine :-)
@timgws For Laravel users that want to help testing.
README.md is missing the usual composer require timgws/query-builder-parser
but this will pull version 1.4 (new tag needed?)
To get the latest, go into composer.json and replace ^1.4
withdev-master
then composer update
yourself to the latest version.
@timgws OK, sounds good!
I accidentally merged #21.
This work was mainly done by @sgjackman @wpanec-uno. Thanks for the contribution peeps!
Please note my comment about dates, this change still needs testing! https://github.com/timgws/QueryBuilderParser/pull/21#issuecomment-340354857