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] - refactor(data/finsupp/basic): split out alist results #18250
Conversation
2e9c9fb
to
10802ed
Compare
These lemmas and definitions are copied without modification.
10802ed
to
8c829d1
Compare
This PR/issue depends on: |
These results will be built upon once I develop the That rant aside, this LGTM. |
Oh absolutely, I'm not saying that the results will never be used; just that they're not likely to be used by anything else in |
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.
I agree this is a useful split. I checked and the only difference introduced is that you haven't opened the finset
and function
namespaces.
maintainer merge
🚀 Pull request has been placed on the maintainer queue by YaelDillies. |
bors r+ |
This file is getting quite long, and nothing else builds upon these results. These lemmas and definitions are copied without modification. They were originally from #15443.
Pull request successfully merged into master. Build succeeded: |
This file is getting quite long, and nothing else builds upon these results.
These lemmas and definitions are copied without modification. They were originally from #15443.
decidable
arguments in lemma statements #18241The first commit is from that PR.