allow passing a method to .default(), for #363#545
Conversation
|
For the record, discussed in chat with @nlf :
We need to find solutions to both before I consider merging. |
|
Just to keep the record up: Functions are now describable, but we still don't have a solution for context. Personally, I lean toward it being too complicated to be worth it, but it's still open for discussion and we're brainstorming solutions. |
|
This implementation now also passes context into the function, but only if the function has a length greater than 0 since this will cause a small performance hit. |
|
@nlf I think we have all we need here, right ? All that's left is documentation, do you mind taking care of it ? Also squashing commits. |
test/any.js
Outdated
There was a problem hiding this comment.
Nitpicking but line break after function.
|
Wrote some docs, let me know if they make sense to you. Once we're happy with those I'll squash the commits and this is ready to go. |
6d50476 to
c8992e1
Compare
|
Updated documentation |
8028443 to
d3a2113
Compare
|
Updated the readme again, I think it's ready now :) |
021f901 to
ad2a1ae
Compare
075434d to
ba40cd5
Compare
allow passing a method to .default(), for #363
|
Almost there. |
|
This thread has been automatically locked due to inactivity. Please open a new issue for related bugs or questions following the new issue template instructions. |
This covers the use cases of
Date.now, UUID generation, and other similar methods.