Thunk return (interface{}, error) in stead of *Result?

graph-gophers / dataloader

Implementation of Facebook's DataLoader in Golang

MIT License

1.21k stars 75 forks source link

Thunk return (interface{}, error) in stead of *Result? #6

Closed nicksrandall closed 7 years ago

nicksrandall commented 7 years ago

I'm thinking about making a small refactor that will change the Thunk and ThunkMany types to return interface{}, and error (or []interface, []error in the case of ThunkMany). I feel like that would be a little more idiomatic.

@tonyghita What do you think?

tonyghita commented 7 years ago

I like this idea 👍

nicksrandall commented 7 years ago

Fixed with #7