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(algebra/group_power/order): When a power is less than one #9700

Closed
wants to merge 3 commits into from

Conversation

YaelDillies
Copy link
Collaborator

@YaelDillies YaelDillies commented Oct 13, 2021

This proves a bunch of simple order lemmas relating 1, a and a ^ n. I also move pow_le_one upstream as it could already be proved in algebra.group_power.order and remove [nontrivial R] from one_lt_pow.


Open in Gitpod

@YaelDillies YaelDillies added the awaiting-review The author would like community review of the PR label Oct 13, 2021
@github-actions github-actions bot added the merge-conflict Please `git merge origin/master` then a bot will remove this label. label Oct 13, 2021
YaelDillies referenced this pull request Oct 13, 2021
…e_lt_pow` (#9696)

`[linear_ordered_semiring R]` becomes `[ordered_semiring R] [nontrivial R]`. I also golf the proof and move ìt from `algebra.field_power` to `algebra.group_power.order`, where it belongs.
@github-actions github-actions bot removed the merge-conflict Please `git merge origin/master` then a bot will remove this label. label Oct 13, 2021
@urkud
Copy link
Member

urkud commented Oct 16, 2021

Otherwise LGTM. Thanks!
bors d+

@bors
Copy link

bors bot commented Oct 16, 2021

✌️ YaelDillies can now approve this pull request. To approve and merge a pull request, simply reply with bors r+. More detailed instructions are available here.

@github-actions github-actions bot added delegated The PR author may merge after reviewing final suggestions. and removed awaiting-review The author would like community review of the PR labels Oct 16, 2021
@YaelDillies
Copy link
Collaborator Author

Thanks for the reviews!

bors merge

bors bot pushed a commit that referenced this pull request Oct 16, 2021
This proves a bunch of simple order lemmas relating `1`, `a` and `a ^ n`. I also move `pow_le_one` upstream as it could already be proved in `algebra.group_power.order` and remove `[nontrivial R]` from `one_lt_pow`.
@bors
Copy link

bors bot commented Oct 16, 2021

Pull request successfully merged into master.

Build succeeded:

@bors bors bot changed the title feat(algebra/group_power/order): When a power is less than one [Merged by Bors] - feat(algebra/group_power/order): When a power is less than one Oct 16, 2021
@bors bors bot closed this Oct 16, 2021
@bors bors bot deleted the pow_le_one branch October 16, 2021 20:32
@eric-wieser eric-wieser added the hacktoberfest-accepted Without this label hacktoberfest is scared off by bors label Oct 26, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
delegated The PR author may merge after reviewing final suggestions. hacktoberfest-accepted Without this label hacktoberfest is scared off by bors
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants