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

feat: RBMap lemmas #739

Merged
merged 3 commits into from
Aug 16, 2024
Merged

feat: RBMap lemmas #739

merged 3 commits into from
Aug 16, 2024

Conversation

digama0
Copy link
Member

@digama0 digama0 commented Apr 13, 2024

This adds RBMap lemmas corresponding to all of the RBSet lemmas.

@digama0 digama0 force-pushed the rbmap_lemmas branch 2 times, most recently from f74f093 to 03d374b Compare April 13, 2024 11:38
@digama0 digama0 added awaiting-review This PR is ready for review; the author thinks it is ready to be merged. depends on another PR This is stacked on top of another Batteries PR. labels Apr 13, 2024
@digama0 digama0 force-pushed the rbmap_lemmas branch 3 times, most recently from 5bf608d to 6201d74 Compare April 17, 2024 09:10
@digama0 digama0 removed the depends on another PR This is stacked on top of another Batteries PR. label Apr 30, 2024
@leanprover-community-mathlib4-bot leanprover-community-mathlib4-bot added the merge-conflict This PR has merge conflicts with the `main` branch which must be resolved by the author. label May 7, 2024
Copy link
Collaborator

@fgdorais fgdorais left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just a few trivial fixes. Otherwise looks good!

Std/Data/RBMap/Lemmas.lean Outdated Show resolved Hide resolved
Std/Data/RBMap/Lemmas.lean Outdated Show resolved Hide resolved
Std/Data/RBMap/Lemmas.lean Outdated Show resolved Hide resolved
digama0 and others added 2 commits August 16, 2024 15:47
Co-authored-by: François G. Dorais <fgdorais@gmail.com>
@digama0 digama0 merged commit f2c939c into main Aug 16, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
awaiting-review This PR is ready for review; the author thinks it is ready to be merged. merge-conflict This PR has merge conflicts with the `main` branch which must be resolved by the author.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants