[Merged by Bors] - feat: port algebra.divisibility.basic#833
[Merged by Bors] - feat: port algebra.divisibility.basic#833
Conversation
This one sounds like it should be fixed wherever the definition is (std?), since that issue is going to keep coming up and there isn't any strong preference for one statement over the other other than easing porting issues. |
Oh actually, I just forgot to check the original mathlib file. Mathport didn't translate The only other change in behavior is the other thing I pointed out. |
|
oh that's bad. Do you have a MWE? (That is, a lean 3 file with a minimum of imports with the syntax that got eaten) |
|
I may not be
Of these, I think this file, Almost all the others look like they are (I should also say: from a cursory glance, it appears that the syntax is eaten up in these other files too) |
|
I finally got oneshot working, and this: -- Insert lean 3 code here.
namespace foo
/-- test -/
theorem foo {A: Type} {a b : A} (h : a = b) : b = a := h^.symm
end footurned into this: -- Insert lean 3 code here.
namespace Foo
/-- test -/
theorem foo {A : Type} {a b : A} (h : a = b) : b = a :=
h
#align foo.foo Foo.foo
end Foo(it lost the |
|
I'm going to assuming Mario is going to deal with the bors d+ |
|
✌️ mpenciak can now approve this pull request. To approve and merge a pull request, simply reply with |
|
Please change the status to |
Co-authored-by: Scott Morrison <scott@tqft.net>
Co-authored-by: Scott Morrison <scott@tqft.net>
Co-authored-by: Scott Morrison <scott@tqft.net>
|
bors r+ |
|
bors r- |
|
bors r+ |
Based off mathlib3 SHA: 70d50ecfd4900dd6d328da39ab7ebd516abe4025
There were a few changes in behavior from Lean 3 to Lean 4:
* `Exists.intro c h` -> `Exists.intro c h.symm` in the proof of `Dvd.intro`
(`h : a * c = b` no longer unifies with `b = a * c`?)
* `dvd_refl` -> `fun {a} => dvd_refl a` in the proof of `dvd_rfl`
(`a | a` no longer unifies with `∀ {a : α}, a ∣ a`?)
Co-authored-by: Matej Penciak <96667244+mpenciak@users.noreply.github.com>
|
Pull request successfully merged into master. Build succeeded: |
Mathlib SHA: e574b1a4e891376b0ef974b926da39e05da12a06 - [x] depends on: #833 Co-authored-by: Matej Penciak <96667244+mpenciak@users.noreply.github.com> Co-authored-by: Joël Riou <joel.riou@universite-paris-saclay.fr> Co-authored-by: Scott Morrison <scott.morrison@gmail.com>
Regenerated from the [port status wiki page](https://github.com/leanprover-community/mathlib/wiki/mathlib4-port-status). Relates to the following PRs: * `algebra.divisibility.basic`: leanprover-community/mathlib4#833 * `algebra.group.with_one.defs`: leanprover-community/mathlib4#841 * `algebra.hom.commute`: leanprover-community/mathlib4#831 * `algebra.hom.group`: leanprover-community/mathlib4#659 * `algebra.hom.units`: leanprover-community/mathlib4#745 * `algebra.ring.basic`: leanprover-community/mathlib4#830 * `category_theory.natural_isomorphism`: leanprover-community/mathlib4#820 * `combinatorics.quiver.connected_component`: leanprover-community/mathlib4#836 * `combinatorics.quiver.subquiver`: leanprover-community/mathlib4#828 Co-authored-by: Johan Commelin <johan@commelin.net>
Based off mathlib3 SHA: 70d50ecfd4900dd6d328da39ab7ebd516abe4025
There were a few changes in behavior from Lean 3 to Lean 4:
Exists.intro c h->Exists.intro c h.symmin the proof ofDvd.intro(
h : a * c = bno longer unifies withb = a * c?)dvd_refl->fun {a} => dvd_refl ain the proof ofdvd_rfl(
a | ano longer unifies with∀ {a : α}, a ∣ a?)