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] - refactor(data/int/gcd,ring_theory/int/basic): collect integer divisibility results from various files #4572
Conversation
…at depend on ring theory and those that don't
Is it ready for review? If yes, please add |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This all looks great to me.
I'm pretty sure that ring_theory/int/basic
shouldn't be too much to import in any of these cases...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Co-authored-by: Johan Commelin <johan@commelin.net>
Co-authored-by: Johan Commelin <johan@commelin.net>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks 🎉
bors merge
…ility results from various files (#4572) Applying comments from PR #4384. In particular: 1) Move the gcd and lcm results from gcd_monoid to `data/int/gcd.lean` with new proofs (for a few lcm results) that do not need ring theory. 2) Try to collect applications of ring theory to ℕ and ℤ into a new file `ring_theory/int/basic.lean`. Co-authored-by: paulvanwamelen <30371019+paulvanwamelen@users.noreply.github.com>
Pull request successfully merged into master. Build succeeded: |
Applying comments from PR #4384. In particular:
data/int/gcd.lean
with new proofs (for a few lcm results) that do not need ring theory.ring_theory/int/basic.lean
.