New issue

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

lift goes up to 11! #1

Closed
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
3 participants
@ivanov

ivanov commented Oct 30, 2014

I got a good laugh out of "lift should go up to 11" at the hackathon in
SF, so decided to submit a PR with the feature

lift goes up to 11!
I got a good laugh out of "lift should go up to 11" at the hackathon in
SF, so decided to submit a PR with the feature
@evancz

This comment has been minimized.

Show comment
Hide comment
@evancz

evancz Nov 3, 2014

Member

Nice :D

Based on the thought process this PR kicked off and other stuff from that meeting, the current plan is as follows:

Core library has Signal.map all the way up to Signal.map5. At that point we'll start recommending (<~) and (~) if you really must go higher.

Part of the logic here is that I don't know of legitimate use cases for N > 5 at this point and we have a safety valve for people who find themselves in that spot. Furthermore, it makes the operators more of a realistic utility instead of a pretty extra.

Member

evancz commented Nov 3, 2014

Nice :D

Based on the thought process this PR kicked off and other stuff from that meeting, the current plan is as follows:

Core library has Signal.map all the way up to Signal.map5. At that point we'll start recommending (<~) and (~) if you really must go higher.

Part of the logic here is that I don't know of legitimate use cases for N > 5 at this point and we have a safety valve for people who find themselves in that spot. Furthermore, it makes the operators more of a realistic utility instead of a pretty extra.

evancz pushed a commit that referenced this pull request Dec 5, 2014

@salutis

This comment has been minimized.

Show comment
Hide comment
@salutis

salutis Feb 4, 2015

Contributor

Do we even need Signal.mapX functions? Wouldn't map, (<~), and (~) be enough?

Contributor

salutis commented Feb 4, 2015

Do we even need Signal.mapX functions? Wouldn't map, (<~), and (~) be enough?

@evancz evancz closed this Feb 6, 2015

@evancz

This comment has been minimized.

Show comment
Hide comment
@evancz

evancz Feb 6, 2015

Member

Having talked to lots of beginners, many people struggle with (<~) and (~) for a while at first. I think it's valuable as an education tool at least, but I personally use mapN all the time.

Member

evancz commented Feb 6, 2015

Having talked to lots of beginners, many people struggle with (<~) and (~) for a while at first. I think it's valuable as an education tool at least, but I personally use mapN all the time.

evancz pushed a commit that referenced this pull request May 26, 2015

Merge pull request #1 from jvoigtlaender/patch-1
make Set.elm compile (it didn't before, because == was not in scope)

jvoigtlaender pushed a commit that referenced this pull request Jan 18, 2016

evancz pushed a commit that referenced this pull request May 17, 2016

Merge pull request #1 from lukewestby/patch-2
add fix err -> badPrimitive from luke
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment