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(Nat/Defs): Add div_dvd_iff_dvd_mul #12243

Closed
wants to merge 4 commits into from

Conversation

Jun2M
Copy link
Collaborator

@Jun2M Jun2M commented Apr 18, 2024


Open in Gitpod

@github-actions github-actions bot added the new-contributor This PR was made by a contributor with fewer than 5 merged PRs. Welcome to the community! label Apr 18, 2024
@Jun2M Jun2M added awaiting-review The author would like community review of the PR easy < 20s of review time. See the lifecycle page for guidelines. and removed easy < 20s of review time. See the lifecycle page for guidelines. labels Apr 18, 2024
@erdOne
Copy link
Member

erdOne commented Apr 20, 2024

maintainer merge

Copy link

🚀 Pull request has been placed on the maintainer queue by erdOne.

rw [← Nat.mul_assoc, Nat.mul_comm _ (a / b), Nat.div_mul_cancel h]
exact Nat.dvd_mul_right a e

@[simp] lemma div_dvd_iff_dvd_mul (h : b ∣ a) (hbpos : 0 < b) : a / b ∣ c ↔ a ∣ b * c :=
Copy link
Collaborator

Choose a reason for hiding this comment

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

We prefer using 0 < b instead of b ≠ 0 when b is a natural.

Suggested change
@[simp] lemma div_dvd_iff_dvd_mul (h : b ∣ a) (hbpos : 0 < b) : a / b ∣ c ↔ a ∣ b * c :=
@[simp] lemma div_dvd_iff_dvd_mul (h : b ∣ a) (hb : b ≠ 0) : a / b ∣ c ↔ a ∣ b * c :=

@YaelDillies
Copy link
Collaborator

maintainer merge

Copy link

🚀 Pull request has been placed on the maintainer queue by YaelDillies.

Copy link
Member

@jcommelin jcommelin left a comment

Choose a reason for hiding this comment

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

Thanks 🎉

bors merge

@leanprover-community-mathlib4-bot leanprover-community-mathlib4-bot added ready-to-merge This PR has been sent to bors. and removed awaiting-review The author would like community review of the PR labels Apr 20, 2024
mathlib-bors bot pushed a commit that referenced this pull request Apr 20, 2024
@mathlib-bors
Copy link

mathlib-bors bot commented Apr 20, 2024

Pull request successfully merged into master.

Build succeeded:

@mathlib-bors mathlib-bors bot changed the title feat(Nat/Defs): Add div_dvd_iff_dvd_mul [Merged by Bors] - feat(Nat/Defs): Add div_dvd_iff_dvd_mul Apr 20, 2024
@mathlib-bors mathlib-bors bot closed this Apr 20, 2024
@mathlib-bors mathlib-bors bot deleted the Jun2M/div_dvd_iff_dvd_mul branch April 20, 2024 21:59
Jun2M added a commit that referenced this pull request Apr 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
maintainer-merge new-contributor This PR was made by a contributor with fewer than 5 merged PRs. Welcome to the community! 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

6 participants