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

sql: add Native value to Type. #104

Merged
merged 1 commit into from
Feb 10, 2017
Merged

sql: add Native value to Type. #104

merged 1 commit into from
Feb 10, 2017

Conversation

smola
Copy link
Contributor

@smola smola commented Jan 27, 2017

Native converts a value of Type into a driver.Value.
This is required for compatibility with the database/sql API.

Native converts a value of Type into a driver.Value.
This is required for compatibility with the database/sql API.
@smola
Copy link
Contributor Author

smola commented Jan 27, 2017

Converting any of our values to a driver.Value is a requirement to implement the standard database/sql interface.

@codecov-io
Copy link

Current coverage is 60.50% (diff: 0.00%)

Merging #104 into master will decrease coverage by 0.51%

@@             master       #104   diff @@
==========================================
  Files            40         40          
  Lines          1652       1666    +14   
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
  Hits           1008       1008          
- Misses          585        599    +14   
  Partials         59         59          

Powered by Codecov. Last update b66aa93...c3b268c

@smola smola merged commit 7179f9f into src-d:master Feb 10, 2017
@smola smola deleted the to-native branch February 10, 2017 09:54
dpordomingo pushed a commit to dpordomingo/gitbase that referenced this pull request Mar 10, 2017
Native converts a value of Type into a driver.Value.
This is required for compatibility with the database/sql API.
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.

3 participants