Closed shaneburrell closed 2 years ago
Hey, @shaneburrell just saw you started a fork. I think i am the last active Maintainer of this package and i would merge your feature now if you want, so no fork is necessary ?
The package updates and PR's were quite a ways behind so I needed to fork as it was holding up use in projects. If development picks back up here I would rather not maintain my own fork and package but for now it was really my only option.
This looks very good to me. I will merge this tomorrow when i am in office and i will release a new minor version then ! No need to fork, i try to look more often into the pull requests now.
Note that since posting the PR I've done quite a bit in my master branch. You may want to just peel that one change.
You mean, i should choose "squash and merge" to keep the git history clean, i assume.
Done so.
The default hard set in code does not work with smalldatetime in MS SQL. This makes it configurable.