Skip to content
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

pass keys to lume.map function #38

Open
solarized-fox opened this issue Oct 3, 2021 · 0 comments · Fixed by NQMVD/lume#4
Open

pass keys to lume.map function #38

solarized-fox opened this issue Oct 3, 2021 · 0 comments · Fixed by NQMVD/lume#4

Comments

@solarized-fox
Copy link

I'm currently in the process of swapping my home-brewed util library for lume. one sticking point is that my version of map passes the value and key to the mapper function, which means I have to modify lume a bit. it should be as simple as changing rtn[k] = f(v) to rtn[k] = f(v, k). I seriously doubt that this would break any existing code (aren't dynamic function arities wonderful?) so if folks think it's a good idea I can make a pull request for this, might also propagate this change to some of the other higher order functions like filter while I'm at it

NQMVD added a commit to NQMVD/lume that referenced this issue Aug 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant