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(to_additive + Cyclic): auto cyclic --> addCyclic #8722

Closed
wants to merge 4 commits into from

Conversation

adomani
Copy link
Collaborator

@adomani adomani commented Nov 29, 2023

Teach the conversion Cyclic ↦ addCyclic to to_additive.


Open in Gitpod

@adomani adomani added the awaiting-review The author would like community review of the PR label Nov 29, 2023
@adomani adomani added the t-meta Tactics, attributes or user commands label Nov 30, 2023
@alreadydone
Copy link
Contributor

Thanks!

maintainer merge

Copy link

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

@RemyDegenne
Copy link
Contributor

Thanks!
bors r+

@github-actions github-actions 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 Nov 30, 2023
mathlib-bors bot pushed a commit that referenced this pull request Nov 30, 2023
Teach the conversion `Cyclic ↦ addCyclic` to `to_additive`.

Affected files:
```bash
GroupTheory/SpecificGroups/Cyclic
Tactic/ToAdditive
```
@eric-wieser eric-wieser changed the title feat(to_additive + Cyclic): auto Cyclic --> addCyclic feat(to_additive + Cyclic): auto cyclic --> addCyclic Nov 30, 2023
@eric-wieser
Copy link
Member

@adomani, is there a reason you've taken to listing the affected files in the commit message? This is tracked automatically by git, so seems redundant (and therefore at risk of being wrong) to me.

@adomani
Copy link
Collaborator Author

adomani commented Nov 30, 2023

Eric, I removed the affected files in this PR. I have a script that creates (a template for) the commit message and had it in there. I'll get rid of it!

@mathlib-bors
Copy link

mathlib-bors bot commented Nov 30, 2023

Pull request successfully merged into master.

Build succeeded:

@mathlib-bors mathlib-bors bot changed the title feat(to_additive + Cyclic): auto cyclic --> addCyclic [Merged by Bors] - feat(to_additive + Cyclic): auto cyclic --> addCyclic Nov 30, 2023
@mathlib-bors mathlib-bors bot closed this Nov 30, 2023
@mathlib-bors mathlib-bors bot deleted the adomani/cyclic_to_additive branch November 30, 2023 09:46
awueth pushed a commit that referenced this pull request Dec 19, 2023
Teach the conversion `Cyclic ↦ addCyclic` to `to_additive`.

Affected files:
```bash
GroupTheory/SpecificGroups/Cyclic
Tactic/ToAdditive
```
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-to-merge This PR has been sent to bors. t-meta Tactics, attributes or user commands
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants