Skip to content
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

Tweaking error messages and supporting IDbConnection implementations … #987

Merged
merged 2 commits into from
Mar 31, 2018
Merged

Tweaking error messages and supporting IDbConnection implementations … #987

merged 2 commits into from
Mar 31, 2018

Conversation

benmccallum
Copy link
Contributor

…that return a DbCommand.

@benmccallum
Copy link
Contributor Author

Extending Nick's error messages with some suggestions. Addresses: #757

@NickCraver
Copy link
Member

Fixed merge, yanking in cleaner create call :)

@NickCraver NickCraver merged commit c6f9e81 into DapperLib:craver/async-errors Mar 31, 2018
NickCraver added a commit that referenced this pull request Apr 4, 2018
…#986)

* Better error messages for .*Async methods when not using  DbConnection

Perhaps in v2 we move these to DbConnection itself ratherr than IDbConnection for the extensions to eliminate the case, but that's breaking...for now we can at least throw a better error. See #757 for details.

* Tweak errors to allow open IDbConnections and those generating DbCommands

Same checks but a bit more lenient with more specific messaging.

* Tweaking error messages and supporting IDbConnection implementations … (#987)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants