-
Notifications
You must be signed in to change notification settings - Fork 381
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] - chore: tidy various files #14968
Conversation
Ruben-VandeVelde
commented
Jul 21, 2024
PR summary 16c5bfa04fImport changes for modified filesNo significant changes to the import graph Import changes for all files
Declarations diffNo declarations were harmed in the making of this PR! 🐙 You can run this locally as follows## summary with just the declaration names:
./scripts/declarations_diff.sh <optional_commit>
## more verbose report:
./scripts/declarations_diff.sh long <optional_commit> |
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.
Looks very nice, as usual - thanks!.
One change I couldn't quite understand, perhaps another reviewer or maintainer does:
maintainer merge
· exact hk | ||
· rw [← @Nat.le_zero, Nat.not_le] at hk | ||
rw [choose_zero_pos R hk] | ||
split_ifs with hk |
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.
Nice golf!
@@ -56,8 +56,10 @@ theorem mem_nonZeroDivisors_of_constantCoeff {φ : MvPowerSeries σ R} | |||
rw [Finset.sum_eq_single (e,0), coeff_zero_eq_constantCoeff] | |||
· rintro ⟨u, _⟩ huv _ | |||
suffices u < e by simp only [he u this, zero_mul, map_zero] | |||
have hue : u ≤ e := by simp only [← mem_antidiagonal.mp huv, le_add_iff_nonneg_right, zero_le] | |||
exact ⟨hue, fun h ↦ by simp_all [mem_antidiagonal.mp huv, le_antisymm hue h]⟩ | |||
apply lt_of_le_of_ne |
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.
I'm not sure if I understand this change: let's have another pair of eyes on it.
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.
It doesn't introduce a new hypothesis, why not.
🚀 Pull request has been placed on the maintainer queue by grunweg. |
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
Pull request successfully merged into master. Build succeeded: |