Stuck on an issue?

Lightrun Answers was designed to reduce the constant googling that comes with debugging 3rd party libraries. It collects links to all the places you might be looking at while hunting down a tough bug.

And, if you’re still stuck at the end, we’re happy to hop on a call to see how we can help out.

possible new implementation?

See original GitHub issue

I have implemented something like daggy here

how is it different?:

Instead of constructor Functions it uses Object.create
when sum type is created it returns type representative and you can call it as it’s just an object
values used to create object is stored as array and there is no need to allocate new array in cata
type definition is stored in type representative and it allows to have only one cata and toStrings

I believe it has a bit better performance too. would like to hear what others think.

Issue Analytics

State:
Created 7 years ago
Reactions:1
Comments:14 (14 by maintainers)

Top GitHub Comments

1reaction

SimonRichardsoncommented, Jan 27, 2017

Move daggy to FL and do 1 imo

On Fri, 27 Jan 2017, 08:08 Irakli Safareli, notifications@github.com wrote:

What would be the best way to move forward on this?

create PR here (it will be a breaking change)

create new repo in FL and create PR there (then create a note here about new implementation)

move daggy to FL and do 1.

@SimonRichardson https://github.com/SimonRichardson

— You are receiving this because you were mentioned.

Reply to this email directly, view it on GitHub https://github.com/puffnfresh/daggy/issues/7#issuecomment-275610285, or mute the thread https://github.com/notifications/unsubscribe-auth/ACcaGFF-0anVFfNI-Rk0STT_uE7vP5pvks5rWaYIgaJpZM4LoSqh .