Skip to content

[Merged by Bors] - feat: port Algebra.Ring.Divisibility#864

Closed
mpenciak wants to merge 6 commits intomasterfrom
mp/algebra-ring-divisibility
Closed

[Merged by Bors] - feat: port Algebra.Ring.Divisibility#864
mpenciak wants to merge 6 commits intomasterfrom
mp/algebra-ring-divisibility

Conversation

@mpenciak
Copy link
Copy Markdown
Collaborator

@mpenciak mpenciak commented Dec 6, 2022

Based off mathlib3 SHA: f1a2caaf51ef593799107fe9a8d5e411599f3996

@mpenciak mpenciak added WIP Work in progress mathlib-port This is a port of a theory file from mathlib. labels Dec 6, 2022
@mpenciak mpenciak changed the title feat: port Algebra.Ring.Divisibility feat: port Algebra.Ring.Divisibility (WIP) Dec 6, 2022
@mpenciak mpenciak removed the WIP Work in progress label Dec 6, 2022
@mpenciak mpenciak changed the title feat: port Algebra.Ring.Divisibility (WIP) feat: port Algebra.Ring.Divisibility Dec 6, 2022
@mpenciak
Copy link
Copy Markdown
Collaborator Author

mpenciak commented Dec 6, 2022

Mathlib3port did a very good job on this file.

Only comment is about the bit0, bit1 lemmas and whether they should be deleted outright or dealt with in another way.


end DistribSemigroup

-- TODO : Delete this?
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
-- TODO : Delete this?
set_option linter.deprecated false in


variable [Ring α] {a b c : α}

-- TODO : Delete this?
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Similarly here.

@kim-em
Copy link
Copy Markdown
Contributor

kim-em commented Dec 6, 2022

bors d+

@bors
Copy link
Copy Markdown

bors bot commented Dec 6, 2022

✌️ mpenciak can now approve this pull request. To approve and merge a pull request, simply reply with bors r+. More detailed instructions are available here.

@github-actions github-actions bot added the delegated This pull request has been delegated to the PR author (or occasionally another non-maintainer). label Dec 6, 2022
mpenciak and others added 2 commits December 6, 2022 09:19
@mpenciak
Copy link
Copy Markdown
Collaborator Author

mpenciak commented Dec 6, 2022

bors r+

bors bot pushed a commit that referenced this pull request Dec 6, 2022
Based off mathlib3 SHA: f1a2caaf51ef593799107fe9a8d5e411599f3996

Co-authored-by: Scott Morrison <scott.morrison@gmail.com>
Co-authored-by: Matej Penciak <96667244+mpenciak@users.noreply.github.com>
@bors
Copy link
Copy Markdown

bors bot commented Dec 6, 2022

Pull request successfully merged into master.

Build succeeded:

@bors bors bot changed the title feat: port Algebra.Ring.Divisibility [Merged by Bors] - feat: port Algebra.Ring.Divisibility Dec 6, 2022
@bors bors bot closed this Dec 6, 2022
@bors bors bot deleted the mp/algebra-ring-divisibility branch December 6, 2022 17:16
bors bot pushed a commit to leanprover-community/mathlib3 that referenced this pull request Dec 7, 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.units`: leanprover-community/mathlib4#848
* `algebra.group.type_tags`: leanprover-community/mathlib4#832
* `algebra.group_with_zero.divisibility`: leanprover-community/mathlib4#870
* `algebra.hom.equiv.basic`: leanprover-community/mathlib4#835
* `algebra.order.group.defs`: leanprover-community/mathlib4#869
* `algebra.order.monoid.basic`: leanprover-community/mathlib4#872
* `algebra.order.monoid.cancel.basic`: leanprover-community/mathlib4#883
* `algebra.order.monoid.with_zero.defs`: leanprover-community/mathlib4#851
* `algebra.order.monoid.with_zero.basic`: leanprover-community/mathlib4#851
* `algebra.ring.divisibility`: leanprover-community/mathlib4#864
* `data.list.defs`: leanprover-community/mathlib4#803
* `data.sigma.order`: leanprover-community/mathlib4#887
* `group_theory.group_action.defs`: leanprover-community/mathlib4#854
* `order.heyting.boundary`: leanprover-community/mathlib4#844
* `order.hom.basic`: leanprover-community/mathlib4#804
* `order.symm_diff`: leanprover-community/mathlib4#842



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

delegated This pull request has been delegated to the PR author (or occasionally another non-maintainer). mathlib-port This is a port of a theory file from mathlib.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants