fix: Allow valid sql types to be used by Columns and Values

[taronish-stytch]

Due to the feature change that allowed nested structs to be passed (#60), two types of data fields were no longer included by calls to Columns and Values that we use:

1. time.Time values
2. structs implementing the driver.Valuer interface

I believe both of these should skip further recursion and be recognized as valid types to be used with this library.

• The sql/driver library recognizes time.Time as valid column type
  • https://cs.opensource.google/go/go/+/refs/tags/go1.20.6:src/database/sql/driver/types.go;l=176
• Golang's sql package specifically maintains conversion for it here
  • https://github.com/golang/go/blob/master/src/database/sql/convert.go#L271-L288
• Implementing driver.Valuer allows for implicit conversion to valid sql statements
  • https://pkg.go.dev/database/sql/driver#Valuer

Happy to talk through any of it, please let me know if you have any feedback. Thanks!

[blockloop]

Hey sorry I've been on vacation all week. Once CI passes I'll merge. Thanks for the PR!

[blockloop]

@taronish-stytch looks like you have a lint issue. I also removed the manual approval process from PR tests. Once you fix the lint issues I'll merge.

[coveralls]

coverage: 99.617% (+0.8%) from 98.8% when pulling 892c834 on taronish-stytch:allow-sql-valid-types into 3e43b34 on blockloop:master.

[blockloop]

v2.2.1 Released