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

Detect column types #6

Merged
merged 5 commits into from Nov 7, 2017

Conversation

Projects
None yet
3 participants
@Songmu
Contributor

Songmu commented Nov 3, 2017

I made the mechanism to detect the SQLite column types, INTEGER, REAL or TEXT.
It is useful for aggregate function use etc.

I did it as default behavior, but I know it is a bit incompatible change, so if you think it it a problem, I can create new command line option for this behavior.

@coveralls

This comment has been minimized.

Show comment
Hide comment
@coveralls

coveralls Nov 3, 2017

Coverage Status

Coverage increased (+1.6%) to 67.492% when pulling d1a6fc9 on Songmu:care-num into 1c5f90f on mattn:master.

coveralls commented Nov 3, 2017

Coverage Status

Coverage increased (+1.6%) to 67.492% when pulling d1a6fc9 on Songmu:care-num into 1c5f90f on mattn:master.

@mattn mattn merged commit e593ec5 into mattn:master Nov 7, 2017

3 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
coverage/coveralls Coverage increased (+1.6%) to 67.492%
Details
wercker/build Wercker pipeline passed
Details
@mattn

This comment has been minimized.

Show comment
Hide comment
@mattn

mattn Nov 7, 2017

Owner

Thank you.

Owner

mattn commented Nov 7, 2017

Thank you.

@Songmu Songmu deleted the Songmu:care-num branch Nov 7, 2017

@Songmu

This comment has been minimized.

Show comment
Hide comment
@Songmu

Songmu Nov 7, 2017

Contributor

Thanks!

Contributor

Songmu commented Nov 7, 2017

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment