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

Add an alterF function to the HashMap module #244

Merged
merged 1 commit into from
Oct 1, 2020

Conversation

aspiwack
Copy link
Member

@aspiwack aspiwack commented Oct 1, 2020

If anything, alterF is part of the optics story (it's a lens in the
style of #197). But it's also generally useful, if rarely so.

Also exposes alter (which has been mysteriously forgotten) and
alterF in the export list.

Depends on #147

If anything, `alterF` is part of the optics story (it's a lens in the
style of #197). But it's also generally useful, if rarely so.

Also exposes `alter` (which has been mysteriously forgotten) and
`alterF` in the export list.
@aspiwack aspiwack requested a review from utdemir October 1, 2020 15:20
@utdemir
Copy link
Contributor

utdemir commented Oct 1, 2020

(Later on we can do the same on vectors and arrays modify functions too, but this PR is good to go)

@utdemir utdemir merged commit 2e4ebb4 into ud/parameter-order-of-linear-collections Oct 1, 2020
@utdemir utdemir deleted the hashmap-alterf branch October 1, 2020 23:35
utdemir pushed a commit that referenced this pull request Oct 2, 2020
If anything, `alterF` is part of the optics story (it's a lens in the
style of #197). But it's also generally useful, if rarely so.

Also exposes `alter` (which has been mysteriously forgotten) and
`alterF` in the export list.
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 this pull request may close these issues.

2 participants