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.

Sign up for GitHub

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

Jump to bottom

default NameMapper #14

Open

foobarhe opened this issue Sep 22, 2017 · 1 comment

foobarhe commented Sep 22, 2017

I believe its better to define the default NameMapper as:
sqlstruct.NameMapper = func (a string) string { return a }

since in most condition, users just want to use the value they defined in the "sql" tag. Not lower them as it is now...

Owner

kisielk commented Dec 5, 2017

Maybe, but I'm not going to change it now.

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