-
Notifications
You must be signed in to change notification settings - Fork 435
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Make SQLServerException constructors public #247
Conversation
…SQLServerBulk record can be implemented outside of the package.
…SQLServerBulk record can be implemented outside of the package.
Merge branch 'dev' into dev
Codecov Report
@@ Coverage Diff @@
## dev #247 +/- ##
============================================
+ Coverage 37.22% 37.23% +<.01%
- Complexity 1662 1668 +6
============================================
Files 103 103
Lines 23673 23673
Branches 3882 3882
============================================
+ Hits 8812 8814 +2
+ Misses 13267 13266 -1
+ Partials 1594 1593 -1
Continue to review full report at Codecov.
|
@mhaderman : Unfortunately your dev branch do not have fixes. Might be some fixes did not pushed to remote dev branch. I am keeping PR open so when you commit changes on DEV branch those commits will be reflect in this PR. |
merge pull from master to dev
@mhaderman : Instead of pushing changes from master to dev can you push changes from your dev to dev branch. May be you need to manually merge only your changes in your dev branch / branch which is forked from mssql-jdbc/dev and then used that branch to create pull request.
Sorry for inconvenience. |
hello @mhaderman , thank you very much for your contribution. We decided to close this PR since the indentation does not match our formatter. However, the implementation you made is great, therefore we created a new PR #334 that is identical to your implementation. Thank you again for your contribution. |
No description provided.