Skip to content

[Merged by Bors] - feat(Algebra/Ring/InjSurj): port file#734

Closed
j-loreaux wants to merge 4 commits intomasterfrom
j-loreaux/algebra.ring.inj_surj
Closed

[Merged by Bors] - feat(Algebra/Ring/InjSurj): port file#734
j-loreaux wants to merge 4 commits intomasterfrom
j-loreaux/algebra.ring.inj_surj

Conversation

@j-loreaux
Copy link
Copy Markdown
Contributor

mathlib3 SHA: 71ca477041bcd6d7c745fe555dc49735c12944b7

porting notes: mostly smooth but:

  1. there were two places where I had to add MulZeroClass explicitly (toward the end of the file), which was weird
  2. one declaration is surprisingly slow for no apparent reason

@j-loreaux j-loreaux added awaiting-review mathlib-port This is a port of a theory file from mathlib. 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
mathlib3 SHA: 71ca477041bcd6d7c745fe555dc49735c12944b7

porting notes: mostly smooth but:

1. there were two places where I had to add `MulZeroClass` explicitly (toward the end of the file), which was weird
2. one declaration is surprisingly slow for no apparent reason
@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(Algebra/Ring/InjSurj): port file [Merged by Bors] - feat(Algebra/Ring/InjSurj): port file Nov 26, 2022
@bors bors bot closed this Nov 26, 2022
@bors bors bot deleted the j-loreaux/algebra.ring.inj_surj branch November 26, 2022 05:46
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.

2 participants