You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
It'd be nice if functions returned {:error, X} and {:ok, Y}. It results in more elegant client code that doesn't have to match against specific structures.
def(%Transmission.Response{id: id), do: ...
vs
def({:ok, %{id: id}}), do: ...
Same same with case and with
The text was updated successfully, but these errors were encountered:
karlseguin
changed the title
More idiomatic API
More idiomatic response tuples
Dec 19, 2016
Thanks for this @karlseguin and sorry for the long latency. This is obviously a breaking change for existing modules, although we do have the beginnings of this in the suppression list module @asgoel very kindly contributed. I suspected at design time that there might be API responses which didn't map cleanly to :ok or :error too and haven't yet bottomed out how to handle those. Will label this for the next major version bump nonetheless.
It'd be nice if functions returned
{:error, X}
and{:ok, Y}
. It results in more elegant client code that doesn't have to match against specific structures.vs
Same same with
case
andwith
The text was updated successfully, but these errors were encountered: