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
[Merged by Bors] - chore(Set/NAry): drop Set.image3
#9221
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
maintainer merge
🚀 Pull request has been placed on the maintainer queue by YaelDillies. |
@YaelDillies, please edit the PR description to explain why we should drop this and what the intended replacement is bors d=@YaelDillies |
✌️ YaelDillies can now approve this pull request. To approve and merge a pull request, simply reply with |
bors merge |
`Set.image3` was only used to prove associativity of `Set.image2`. It had basically no API and had exactly one (easily replaced) use outside `Data.Set.NAry`). There is no specific function as a replacement, but it can be obtained by combining `Set.image2` twice.
Pull request successfully merged into master. Build succeeded: |
Set.image3
Set.image3
Set.image3
was only used to prove associativity ofSet.image2
. It had basically no API and had exactly one (easily replaced) use outsideData.Set.NAry
). There is no specific function as a replacement, but it can be obtained by combiningSet.image2
twice.As requested by @YaelDillies