-
Notifications
You must be signed in to change notification settings - Fork 298
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(algebra/group_with_zero_power): review #2966
Conversation
This was a copy of `fpow_neg_one` with a misleading name.
fpow_inv
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
List of changes: * Rename `gpow_neg_succ` to `gpow_neg_succ_of_nat` to match other names in `int` namespace. * Add `units.coe_gpow`. * Remove `fpow_neg_succ`, leave `fpow_neg_succ_of_nat`. * Rewrite the proof of `fpow_add` in the same way I rewrote the proof of `gpow_add`. * Make argument `a` implicit in some lemmas because they have an argument `ha : a ≠ 0`. * Remove `fpow_inv`. This was a copy of `fpow_neg_one` with a misleading name. * Remove `unit_pow` in favor of a more general `units.coe_pow`. * Remove `unit_gpow`, add a more general `units.coe_gpow'` instead.
bors r- |
Build fails, so I removed this from the queue. |
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.
bors r+
bors merge |
Already running a review |
List of changes: * Rename `gpow_neg_succ` to `gpow_neg_succ_of_nat` to match other names in `int` namespace. * Add `units.coe_gpow`. * Remove `fpow_neg_succ`, leave `fpow_neg_succ_of_nat`. * Rewrite the proof of `fpow_add` in the same way I rewrote the proof of `gpow_add`. * Make argument `a` implicit in some lemmas because they have an argument `ha : a ≠ 0`. * Remove `fpow_inv`. This was a copy of `fpow_neg_one` with a misleading name. * Remove `unit_pow` in favor of a more general `units.coe_pow`. * Remove `unit_gpow`, add a more general `units.coe_gpow'` instead.
Pull request successfully merged into master. Build succeeded: |
List of changes: * Rename `gpow_neg_succ` to `gpow_neg_succ_of_nat` to match other names in `int` namespace. * Add `units.coe_gpow`. * Remove `fpow_neg_succ`, leave `fpow_neg_succ_of_nat`. * Rewrite the proof of `fpow_add` in the same way I rewrote the proof of `gpow_add`. * Make argument `a` implicit in some lemmas because they have an argument `ha : a ≠ 0`. * Remove `fpow_inv`. This was a copy of `fpow_neg_one` with a misleading name. * Remove `unit_pow` in favor of a more general `units.coe_pow`. * Remove `unit_gpow`, add a more general `units.coe_gpow'` instead.
List of changes:
gpow_neg_succ
togpow_neg_succ_of_nat
to match other names inint
namespace.units.coe_gpow
.fpow_neg_succ
, leavefpow_neg_succ_of_nat
.fpow_add
in the same way I rewrote the proof ofgpow_add
.a
implicit in some lemmas because they have an argumentha : a ≠ 0
.fpow_inv
. This was a copy offpow_neg_one
with a misleading name.unit_pow
in favor of a more generalunits.coe_pow
.unit_gpow
, add a more generalunits.coe_gpow'
instead.