Skip to content

[Merged by Bors] - Chore(Algebra/Polynomial/Module/Basic): Refactor out Module.AEval #13613

Closed
Shamrock-Frost wants to merge 4 commits intomasterfrom
AEval-refactor
Closed

[Merged by Bors] - Chore(Algebra/Polynomial/Module/Basic): Refactor out Module.AEval #13613
Shamrock-Frost wants to merge 4 commits intomasterfrom
AEval-refactor

Conversation

@Shamrock-Frost
Copy link
Copy Markdown
Contributor

Refactor Module.AEval out of Algebra/Polynomial/Module/Basic.


See #13573 for previous discussion

Open in Gitpod

@Shamrock-Frost Shamrock-Frost added awaiting-review awaiting-CI This PR does not pass CI yet. This label is automatically removed once it does. t-algebra Algebra (groups, rings, fields, etc) labels Jun 7, 2024
@ghost ghost added the merge-conflict The PR has a merge conflict with master, and needs manual merging. (this label is managed by a bot) label Jun 7, 2024
@github-actions
Copy link
Copy Markdown

github-actions bot commented Jun 7, 2024

PR summary

Import changes

No significant changes to the import graph


Declarations diff

No declarations were harmed in the making of this PR! 🐙

You can run this locally as follows
## summary with just the declaration names:
./scripts/no_lost_declarations.sh short <optional_commit>

## more verbose report:
./scripts/no_lost_declarations.sh <optional_commit>

@ghost ghost removed the merge-conflict The PR has a merge conflict with master, and needs manual merging. (this label is managed by a bot) label Jun 7, 2024
@github-actions github-actions bot removed the awaiting-CI This PR does not pass CI yet. This label is automatically removed once it does. label Jun 7, 2024
@erdOne
Copy link
Copy Markdown
Member

erdOne commented Jun 8, 2024

LGTM otherwise. Thanks!

Co-authored-by: Andrew Yang <36414270+erdOne@users.noreply.github.com>
@erdOne
Copy link
Copy Markdown
Member

erdOne commented Jun 8, 2024

maintainer merge

@github-actions
Copy link
Copy Markdown

github-actions bot commented Jun 8, 2024

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

@github-actions github-actions bot added the maintainer-merge A reviewer has approved the changed; awaiting maintainer approval. label Jun 8, 2024
Copy link
Copy Markdown
Member

@jcommelin jcommelin left a comment

Choose a reason for hiding this comment

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

Thanks 🎉

bors merge

@ghost ghost added ready-to-merge This PR has been sent to bors. and removed awaiting-review labels Jun 8, 2024
mathlib-bors bot pushed a commit that referenced this pull request Jun 8, 2024
…3613)

Refactor Module.AEval out of Algebra/Polynomial/Module/Basic.
@mathlib-bors
Copy link
Copy Markdown
Contributor

mathlib-bors bot commented Jun 8, 2024

Pull request successfully merged into master.

Build succeeded:

@mathlib-bors mathlib-bors bot changed the title Chore(Algebra/Polynomial/Module/Basic): Refactor out Module.AEval [Merged by Bors] - Chore(Algebra/Polynomial/Module/Basic): Refactor out Module.AEval Jun 8, 2024
@mathlib-bors mathlib-bors bot closed this Jun 8, 2024
@mathlib-bors mathlib-bors bot deleted the AEval-refactor branch June 8, 2024 19:30
AntoineChambert-Loir pushed a commit that referenced this pull request Jun 20, 2024
…3613)

Refactor Module.AEval out of Algebra/Polynomial/Module/Basic.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

maintainer-merge A reviewer has approved the changed; awaiting maintainer approval. ready-to-merge This PR has been sent to bors. t-algebra Algebra (groups, rings, fields, etc)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants