Skip to content
This repository has been archived by the owner on Feb 13, 2019. It is now read-only.

Make it so SqlServerCommandProvider accepts TKey argument #3

Closed
adamfoneil opened this issue Jun 25, 2018 · 0 comments
Closed

Make it so SqlServerCommandProvider accepts TKey argument #3

adamfoneil opened this issue Jun 25, 2018 · 0 comments

Comments

@adamfoneil
Copy link
Owner

This way, the SQL syntax and key type isn't coupled -- there's no reason for them to be.
Then add extension classes from there that target different key type flavors: SqlServerIntProvider, SqlServerGuidProvider, SqlServerLongProvider, etc.
adamfoneil added a commit that referenced this issue Jun 25, 2018
@adamfoneil
#3 looks pretty good, but need some tests
a901ce9
adamfoneil added a commit that referenced this issue Jun 25, 2018
@adamfoneil
Merge pull request #4 from /issues/3
b12584a 
Issues/3
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@adamfoneil