-
Notifications
You must be signed in to change notification settings - Fork 178
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
Add filterKeys
#866
Comments
|
alright, but what about the addition of |
I don't feel strongly about it. I guess it adds symmetry, balancing |
I'll go do the needful then. Additionally, I'm finding somethings are used internally, with full haddocks that would be useful to export, but are not (e.g. |
|
I'll raise separate PRs to not dirty the history |
Hi All,
A more general form of
restrictKeys
would befilterKeys
, implemented thusly:Which would allow
restrictKeys
to be implemented simply asIs there an appetite for this kind of change?
filterKeys
is a pretty generally useful function, andrestrictKeys
tends to be a little too specialized too suit my needs. Since the change would arguably improve the existing functions, I think it's worth a thought.The text was updated successfully, but these errors were encountered: