Closed avibryant closed 9 years ago
yeah, this feels less prescriptive which I think is a good thing.
On Monday, December 29, 2014, Avi Bryant notifications@github.com wrote:
This is a much simpler but less satisfying version of #18 https://github.com/stripe/brushfire/pull/18, which just lets you wrap an Evaluator around an Error, without actually eliminating the Evaluator trait.
cc @snoble https://github.com/snoble
You can merge this Pull Request by running
git pull https://github.com/stripe/brushfire avi-error-evaluator
Or view, comment on, or merge it at:
https://github.com/stripe/brushfire/pull/27 Commit Summary
- ErrorEvaluator
File Changes
- M src/main/scala/com/stripe/brushfire/Evaluators.scala https://github.com/stripe/brushfire/pull/27/files#diff-0 (15)
Patch Links:
— Reply to this email directly or view it on GitHub https://github.com/stripe/brushfire/pull/27.
This is a much simpler but less satisfying version of https://github.com/stripe/brushfire/pull/18, which just lets you wrap an Evaluator around an Error, without actually eliminating the Evaluator trait.
cc @snoble