glerchundi / sqlboiler-crdb

23 stars 20 forks source link

Use stdlib null types #35

Closed lopezator closed 3 years ago

lopezator commented 4 years ago

As of Go 1.13 new null types have been added to the stdlib.

https://golang.org/pkg/database/sql/#NullTime

I think relying on them instead of github.com/volatiletech/null/v8 whenever possible would be a good idea, so users working with these types don't have to rely on a third-party library.

glerchundi commented 3 years ago

As the generated code already uses the nullable package from volatiletech, although it's not a requirement to make everything work, I would like to keep it on the generator code as well. Seems more consistent IMO.