If GetAll finds no results in the database it is currently a no-op. In the interests of safety it would be better if this function returned the ErrNoRows error. If no results is not an error case the user can explicitly match against and ignore the error.
If
GetAllfinds no results in the database it is currently a no-op. In the interests of safety it would be better if this function returned theErrNoRowserror. If no results is not an error case the user can explicitly match against and ignore the error.