We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Current implementation:
trial/trial/src/Trial.hs
Lines 298 to 301 in 49ad9a0
Proposal:
(<|>) :: Trial e a -> Trial e a -> Trial e a r@Result{} <|> _ = r f@Fiasco{} <|> r@Result{} = f <> r (Fiasco e1) <|> (Fiasco e2) = Fiasco (e1 <> e2)
New semantics: return first Result, but also accumulate all previous Fiascos.
Result
Fiasco
The text was updated successfully, but these errors were encountered:
Let's first check is the new semantic would preserve all the laws. If not we still can add such function as I see how useful it could be sometimes 🙂
Sorry, something went wrong.
[#50] Change Alternative instance
9d3a202
Resolves #50
[#50] Change Alternative instance (#51)
5100c3f
* [#50] Change Alternative instance Resolves #50 * Update trial/src/Trial.hs Co-authored-by: Veronika Romashkina <[email protected]> * Fix after review Co-authored-by: Veronika Romashkina <[email protected]>
chshersh
Successfully merging a pull request may close this issue.
Current implementation:
trial/trial/src/Trial.hs
Lines 298 to 301 in 49ad9a0
Proposal:
New semantics: return first
Result
, but also accumulate all previousFiasco
s.The text was updated successfully, but these errors were encountered: