Join GitHub today
GitHub is home to over 28 million developers working together to host and review code, manage projects, and build software together.
Sign upImproved documentation of Signal.foldp #149
Conversation
evancz
merged commit 6aabf51
into
elm:master
Jan 29, 2015
1 check failed
continuous-integration/travis-ci
The Travis CI build could not complete due to an error
Details
jvoigtlaender
deleted the
jvoigtlaender:docu-signal-module
branch
Jan 29, 2015
This comment has been minimized.
Show comment
Hide comment
This comment has been minimized.
Show comment
Hide comment
mietek
commented
Dec 31, 2015
|
What happened to this PR? I don’t see the commit in the history of |
jvoigtlaender
restored the
jvoigtlaender:docu-signal-module
branch
Dec 31, 2015
This comment has been minimized.
Show comment
Hide comment
This comment has been minimized.
Show comment
Hide comment
jvoigtlaender
Dec 31, 2015
Contributor
This is a very good question. @evancz, you merged this PR at some time, but nevertheless it, and its effect, are not visible. Did you mess with the git history at some point?
The change is still a useful one, I think. Should I simply make a new PR with the same content?
|
This is a very good question. @evancz, you merged this PR at some time, but nevertheless it, and its effect, are not visible. Did you mess with the git history at some point? The change is still a useful one, I think. Should I simply make a new PR with the same content? |
jvoigtlaender
referenced this pull request
Jan 6, 2016
Merged
Improved documentation of Signal.foldp #475
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
jvoigtlaender commentedJan 29, 2015
Meant to keep issues/surprises/questions like https://github.com/elm-lang/core/issues/148 and https://groups.google.com/d/msg/elm-discuss/m4JB7CPXbLI/Dbac1AaAmd0J from arising.