Skip to content
This repository was archived by the owner on Jul 24, 2024. It is now read-only.

[Merged by Bors] - chore(group_theory/group_action/group): add smul_inv#7568

Closed
eric-wieser wants to merge 2 commits intomasterfrom
eric-wieser/smul_inv
Closed

[Merged by Bors] - chore(group_theory/group_action/group): add smul_inv#7568
eric-wieser wants to merge 2 commits intomasterfrom
eric-wieser/smul_inv

Conversation

@eric-wieser
Copy link
Copy Markdown
Member

This renames the existing smul_inv to smul_inv', which is consistent with the name of the other lemma about mul_semiring_action, smul_mul'.


Open in Gitpod

This renames the existing `smul_inv` to `smul_inv'`, which is consistent with the name of the other lemma about `mul_semiring_action`, `smul_mul'`.
@eric-wieser eric-wieser added easy < 20s of review time. See the lifecycle page for guidelines. awaiting-review The author would like community review of the PR labels May 10, 2021
Comment on lines +105 to +107
lemma smul_inv [group β] [smul_comm_class α β β] [is_scalar_tower α β β] (c : α) (x : β) :
(c • x)⁻¹ = c⁻¹ • x⁻¹ :=
by rw [inv_eq_iff_mul_eq_one, smul_mul_smul, mul_right_inv, mul_right_inv, one_smul]
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

This name was chosen to be consistent with mul_inv : (c * x)⁻¹ = c⁻¹ * x⁻¹

@fpvandoorn
Copy link
Copy Markdown
Member

LGTM. A bunch of mul_semiring_action lemmas are already primed anyway.

bors merge

@github-actions github-actions bot added ready-to-merge All that is left is for bors to build and merge this PR. (Remember you need to say `bors r+`.) and removed awaiting-review The author would like community review of the PR labels May 15, 2021
bors bot pushed a commit that referenced this pull request May 15, 2021
This renames the existing `smul_inv` to `smul_inv'`, which is consistent with the name of the other lemma about `mul_semiring_action`, `smul_mul'`.
@bors
Copy link
Copy Markdown

bors bot commented May 16, 2021

Pull request successfully merged into master.

Build succeeded:

@bors bors bot changed the title chore(group_theory/group_action/group): add smul_inv [Merged by Bors] - chore(group_theory/group_action/group): add smul_inv May 16, 2021
@bors bors bot closed this May 16, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

easy < 20s of review time. See the lifecycle page for guidelines. ready-to-merge All that is left is for bors to build and merge this PR. (Remember you need to say `bors r+`.)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants