search

lttb / flown

Type Helpers for Flow (static typechecker)
http://npm.im/flown
MIT License
85 stars 7 forks source link

Current path implementation brokes with same path length because flow binds given path #5

Closed ilya-m32 closed 5 years ago

ilya-m32 commented 5 years ago

Error example (implementation was copied from Path is here.

Works fine with previous implementation (which was commented due to multiple declare, I guess?)

ilya-m32 commented 5 years ago

cast @lttb

ilya-m32 commented 5 years ago

actually, second impl does not work for longer paths either Here is an example

So far I have come to stupid but working (with all my additional tests) solution (_Path is curried and has reversed arguments, but it can be changed easily)

lttb commented 5 years ago

Thank you for the issue and fix!

Fixed in https://github.com/lttb/flown/pull/6 Released in v0.2.1