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

[Merged by Bors] - feat: port Algebra.Hom.Group #659

Closed
wants to merge 102 commits into from
Closed

Conversation

winstonyin
Copy link
Collaborator

@winstonyin winstonyin commented Nov 20, 2022

@winstonyin winstonyin added the WIP Work in progress label Nov 20, 2022
@semorrison
Copy link
Contributor

That's fixed, but now blocked by leanprover/lean4#1907.

@ChrisHughes24 ChrisHughes24 removed the blocked-by-core-PR This PR depends on a PR to Core/Std label Dec 2, 2022
@semorrison semorrison added the blocked-by-core-PR This PR depends on a PR to Core/Std label Dec 2, 2022
@semorrison
Copy link
Contributor

semorrison commented Dec 2, 2022

I think I've now killed off all the porting notes (baring one about to_additive).

This PR uses a forked version of Lean 4 until nightly-2022-12-03 is available. It will be broken until aesop gets updated in leanprover-community/aesop#32. (Edit: aesop updated.)

@kbuzzard kbuzzard mentioned this pull request Dec 2, 2022
2 tasks
@semorrison semorrison added the merge-conflict The PR has a merge conflict with master, and needs manual merging. label Dec 2, 2022
@semorrison
Copy link
Contributor

bors merge

@bors
Copy link

bors bot commented Dec 2, 2022

👎 Rejected by label

@github-actions github-actions bot added ready-to-merge This PR has been sent to bors. and removed awaiting-review labels Dec 2, 2022
@semorrison semorrison removed merge-conflict The PR has a merge conflict with master, and needs manual merging. blocked-by-core-PR This PR depends on a PR to Core/Std labels Dec 2, 2022
@semorrison
Copy link
Contributor

bors merge

bors bot pushed a commit that referenced this pull request Dec 2, 2022
mathlib3 SHA: 8c53048add6ffacdda0b36c4917bfe37e209b0ba

- [x] depends on #563
- [x] depends on leanprover-community/mathlib#17733
- [x] depends on leanprover/lean4#1901
- [x] depends on leanprover/lean4#1907

Co-authored-by: Winston Yin <winstonyin@gmail.com>
Co-authored-by: Scott Morrison <scott.morrison@gmail.com>
Co-authored-by: Jireh Loreaux <loreaujy@gmail.com>
Co-authored-by: Ruben Van de Velde <65514131+Ruben-VandeVelde@users.noreply.github.com>
@bors
Copy link

bors bot commented Dec 2, 2022

Pull request successfully merged into master.

Build succeeded:

@bors bors bot changed the title feat: port Algebra.Hom.Group [Merged by Bors] - feat: port Algebra.Hom.Group Dec 2, 2022
@bors bors bot closed this Dec 2, 2022
@bors bors bot deleted the hom_group branch December 2, 2022 23:45
bors bot pushed a commit that referenced this pull request Dec 3, 2022
mathlib3 SHA: 76171581280d5b5d1e2d1f4f37e5420357bdc636

Seems relatively straightforward. Some lemma names (e.g. `units.coe_zpow -> Units.val_zpow_eq_zpow_val`) are renamed to refer to `val` instead of `coe`, following the convention used in `Algebra.Group.Units`.

- [x] depends on: #659

Co-authored-by: Winston Yin <winstonyin@gmail.com>
Co-authored-by: Scott Morrison <scott.morrison@gmail.com>
Co-authored-by: Jireh Loreaux <loreaujy@gmail.com>
Co-authored-by: Ruben Van de Velde <65514131+Ruben-VandeVelde@users.noreply.github.com>
bors bot pushed a commit to leanprover-community/mathlib that referenced this pull request Dec 6, 2022
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>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
mathlib-port This is a port of a theory file from mathlib. ready-to-merge This PR has been sent to bors.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

8 participants