Skip to content

[Merged by Bors] - feat: port algebra.group.semiconj#717

Closed
dwarn wants to merge 7 commits intomasterfrom
group_semiconj
Closed

[Merged by Bors] - feat: port algebra.group.semiconj#717
dwarn wants to merge 7 commits intomasterfrom
group_semiconj

Conversation

@dwarn
Copy link
Copy Markdown
Contributor

@dwarn dwarn commented Nov 24, 2022

mathlib SHA e50b8c261b0a000b806ec0e1356b41945eda61f7

This completes an earlier partial port of the file.

I added a workaround for the issue explained by Floris at #707 (comment).

@dwarn dwarn added mathlib-port This is a port of a theory file from mathlib. awaiting-review labels Nov 24, 2022
@urkud
Copy link
Copy Markdown
Member

urkud commented Nov 24, 2022

I see that the mathlib3port file has a lot of "dubious translation" warnings. What do they mean?

bors bot pushed a commit that referenced this pull request Nov 24, 2022
* Remove instance and add `to_additive`
* Point some comments to the right issue
* Resolves the issue in #707
* Might conflict with #717
@Ruben-VandeVelde
Copy link
Copy Markdown
Contributor

maintainer merge

@kim-em
Copy link
Copy Markdown
Contributor

kim-em commented Nov 25, 2022

You seem to not be authorized

@Ruben-VandeVelde
Copy link
Copy Markdown
Contributor

Oh, I may have misunderstood how that worked. Anyway, this one seems ready to go

@kim-em
Copy link
Copy Markdown
Contributor

kim-em commented Nov 25, 2022

@Ruben-VandeVelde, sorry, the maintainer merge bot was misconfigured. Hopefully it will work now.

@kim-em kim-em added awaiting-author A reviewer has asked the author a question or requested changes. and removed awaiting-review labels Nov 25, 2022
@dwarn dwarn added awaiting-review and removed awaiting-author A reviewer has asked the author a question or requested changes. labels Nov 26, 2022
@kim-em
Copy link
Copy Markdown
Contributor

kim-em commented Nov 26, 2022

bors merge

@github-actions github-actions bot added ready-to-merge This PR has been sent to bors. and removed awaiting-review labels Nov 26, 2022
bors bot pushed a commit that referenced this pull request Nov 26, 2022
mathlib SHA e50b8c261b0a000b806ec0e1356b41945eda61f7

This completes an earlier partial port of the file.

~~I added a workaround for the issue explained by Floris at #707 (comment)

Co-authored-by: Ruben Van de Velde <65514131+Ruben-VandeVelde@users.noreply.github.com>
@bors
Copy link
Copy Markdown

bors bot commented Nov 26, 2022

Pull request successfully merged into master.

Build succeeded:

@bors bors bot changed the title feat: port algebra.group.semiconj [Merged by Bors] - feat: port algebra.group.semiconj Nov 26, 2022
@bors bors bot closed this Nov 26, 2022
@bors bors bot deleted the group_semiconj branch November 26, 2022 16:55
bors bot pushed a commit to leanprover-community/mathlib3 that referenced this pull request Nov 27, 2022
Regenerated from the [port status wiki page](https://github.com/leanprover-community/mathlib/wiki/mathlib4-port-status).
Relates to the following PRs:
* `algebra.group.semiconj`: leanprover-community/mathlib4#717
* `algebra.group_with_zero.basic`: leanprover-community/mathlib4#669
* `algebra.group_with_zero.inj_surj`: leanprover-community/mathlib4#722
* `algebra.ring.inj_surj`: leanprover-community/mathlib4#734
* `algebra.ring.units`: leanprover-community/mathlib4#746
* `data.finite.defs`: leanprover-community/mathlib4#698
* `order.lattice`: leanprover-community/mathlib4#642
rosborn added a commit that referenced this pull request Nov 27, 2022
* master:
  feat: port Data.Countable.Defs (#736)
  chore: bump to nightly-2022-11-26 (#747)
  feat(Algebra/Ring/Units): port file (#746)
  style(Algebra/Order/Monoid/Lemmas): Update to current naming convention (#743)
  feat: stubs in ad-hoc ported files for linarith algebra requirements (#733)
  feat: port algebra.group.semiconj (#717)
  chore: make argument to mul_inv_cancel implicit (#737)
  feat(Algebra/Ring/InjSurj): port file (#734)
  chore: bump to nightly-2022-11-25 (#731)
  feat: port order.minmax (#728)
  chore: make the 'p' argument to Nat.find implicit (#730)
  feat: port Control.Basic (#588)
  feat: port data.finite.defs (#698)
  feat: port:  Data.DList.Basic (#616)
  chore: reduce imports in Algebra.Group.Defs (#727)
  chore(Algebra/Order/Hom/Basic): remove outParam (#692)
bors bot pushed a commit to leanprover-community/mathlib3 that referenced this pull request Nov 27, 2022
Regenerated from the [port status wiki page](https://github.com/leanprover-community/mathlib/wiki/mathlib4-port-status).
Relates to the following PRs:
* `algebra.group.semiconj`: leanprover-community/mathlib4#717
* `algebra.group_with_zero.basic`: leanprover-community/mathlib4#669
* `algebra.group_with_zero.inj_surj`: leanprover-community/mathlib4#722
* `algebra.ring.inj_surj`: leanprover-community/mathlib4#734
* `algebra.ring.units`: leanprover-community/mathlib4#746
* `data.finite.defs`: leanprover-community/mathlib4#698
* `order.lattice`: leanprover-community/mathlib4#642
bors bot pushed a commit to leanprover-community/mathlib3 that referenced this pull request Nov 28, 2022
Regenerated from the [port status wiki page](https://github.com/leanprover-community/mathlib/wiki/mathlib4-port-status).
Relates to the following PRs:
* `algebra.group.semiconj`: leanprover-community/mathlib4#717
* `algebra.group_with_zero.basic`: leanprover-community/mathlib4#669
* `algebra.group_with_zero.inj_surj`: leanprover-community/mathlib4#722
* `algebra.ring.inj_surj`: leanprover-community/mathlib4#734
* `algebra.ring.units`: leanprover-community/mathlib4#746
* `data.finite.defs`: leanprover-community/mathlib4#698
* `order.lattice`: leanprover-community/mathlib4#642
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.

4 participants