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] - feat: port Algebra.Module.Pi #1283
Conversation
One of the errors will be fixed after leanprover/lean4@70a6c06 makes it downstream to mathlib. Might as well wait and see if all of them are. |
This PR/issue depends on:
|
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.
bors d+
✌️ hrmacbeth can now approve this pull request. To approve and merge a pull request, simply reply with |
Co-authored-by: Johan Commelin <johan@commelin.net>
bors r+ |
One earlier failure was extensively discussed: - https://leanprover.zulipchat.com/#narrow/stream/287929-mathlib4/topic/Performance.20issue.20with.20.60CompleteBooleanAlgebra.60/near/319019205 - https://leanprover.zulipchat.com/#narrow/stream/270676-lean4/topic/type.20class.20inference.20looping and is fixed as of leanprover/lean4@70a6c06, in mathlib as of the bump #1335. Another failure posted to Zulip and the Lean 4 repo - https://leanprover.zulipchat.com/#narrow/stream/270676-lean4/topic/type.20class.20inference.20failure.20in.20pi.20type - leanprover/lean4#2011 and is fixed as of leanprover/lean4@fedf235, in mathlib as of the bump #1397. There is one more mysterious `apply` failure, now worked around; we should track this down someday. - [x] depends on: #1397 Co-authored-by: Scott Morrison <scott.morrison@gmail.com>
Pull request successfully merged into master. Build succeeded:
|
One earlier failure was extensively discussed: - https://leanprover.zulipchat.com/#narrow/stream/287929-mathlib4/topic/Performance.20issue.20with.20.60CompleteBooleanAlgebra.60/near/319019205 - https://leanprover.zulipchat.com/#narrow/stream/270676-lean4/topic/type.20class.20inference.20looping and is fixed as of leanprover/lean4@70a6c06, in mathlib as of the bump #1335. Another failure posted to Zulip and the Lean 4 repo - https://leanprover.zulipchat.com/#narrow/stream/270676-lean4/topic/type.20class.20inference.20failure.20in.20pi.20type - leanprover/lean4#2011 and is fixed as of leanprover/lean4@fedf235, in mathlib as of the bump #1397. There is one more mysterious `apply` failure, now worked around; we should track this down someday. - [x] depends on: #1397 Co-authored-by: Scott Morrison <scott.morrison@gmail.com>
One earlier failure was extensively discussed:
and is fixed as of leanprover/lean4@70a6c06, in mathlib as of the bump #1335.
Another failure posted to Zulip and the Lean 4 repo
and is fixed as of leanprover/lean4@fedf235, in mathlib as of the bump #1397.
There is one more mysterious
apply
failure, now worked around; we should track this down someday.