search

mgutz / dat

Go Postgres Data Access Toolkit
Other
612 stars 62 forks source link

Remove dat.ErrNotFound #37

Open sbowman opened 8 years ago

sbowman commented 8 years ago

Half the query functions in dat use the standard "sql.ErrNoRows" and half use "dat.ErrNotFound". Could you please just standardize on "sql.ErrNoRows" and drop "dat.ErrNotFound", so we can stop having to guess which is which or testing for both all the time?

mgutz commented 8 years ago

I agree with this. dat used to work with another postgres driver (pgx) and I was being too clever in my abstractions. The next version will remove it. I can't remove it from the current version as it will break API compatibility.