switch order of arguments to R.repeatN#626
Conversation
|
👍 🌵 |
switch order of arguments to R.repeatN
|
👎 Unlike the others on this list, this is the only one that actually makes sense curried, And if it's used that way, I would guess that @davidchambers: You've been our strongest advocate for consistency, and I admire that. But I think there is a more important consistency here than "where parameter |
|
Damn, A few seconds late. Sorry, I've been both sick, and lacking Internet, managing to see a few issues on my phone, but not really to engage. |
|
In that case, how about we revert to the old behaviour but rename the function |
|
No objections. I like your focus on naming consistency; it's kind of like having JDD around to poke us on performance here and there: While I know you're almost always thinking about these things right, it's often not something I would have thought about myself. 👍 |
Every other xxxN function takes the N as the first argument:
I can't imagine partially applying
R.repeatNvery often. (Please correct me if I'm wrong!) This being the case, we should switch the argument order so all the xxxN functions take the N as the first argument.