Skip to content

Remove dat.ErrNotFound #37

Open
Open

Description

@sbowman

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?

Activity

  1. mgutz commented on Mar 1, 2016

    @mgutz
    Owner

    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.

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

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions