Skip to content

[Merged by Bors] - feat(RingTheory/Flat/Basic): if a subalgebra is flat, then it's also flat as a submodule#15360

Closed
acmepjz wants to merge 1 commit intomasterfrom
acmepjz_flat_tosubmodule
Closed

[Merged by Bors] - feat(RingTheory/Flat/Basic): if a subalgebra is flat, then it's also flat as a submodule#15360
acmepjz wants to merge 1 commit intomasterfrom
acmepjz_flat_tosubmodule

Conversation

@acmepjz
Copy link
Copy Markdown
Collaborator

@acmepjz acmepjz commented Jul 31, 2024


This is suggested in #15346 (comment).

I don't know if it is easy since it adds an instance, which may have performance impact on the whole mathlib.

Open in Gitpod

@acmepjz acmepjz added easy < 20s of review time. See the lifecycle page for guidelines. 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 Jul 31, 2024
@github-actions
Copy link
Copy Markdown

PR summary d118d36d76

Import changes for modified files

No significant changes to the import graph

Import changes for all files
Files Import difference

Declarations diff

+ instSubalgebraToSubmodule

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

## more verbose report:
./scripts/declarations_diff.sh long <optional_commit>
The doc-module for `script/declarations_diff.sh` contains some details about this script.

@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 Jul 31, 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

mathlib-bors bot pushed a commit that referenced this pull request Aug 22, 2024
@mathlib-bors
Copy link
Copy Markdown
Contributor

mathlib-bors bot commented Aug 22, 2024

Pull request successfully merged into master.

Build succeeded:

@mathlib-bors mathlib-bors bot changed the title feat(RingTheory/Flat/Basic): if a subalgebra is flat, then it's also flat as a submodule [Merged by Bors] - feat(RingTheory/Flat/Basic): if a subalgebra is flat, then it's also flat as a submodule Aug 22, 2024
@mathlib-bors mathlib-bors bot closed this Aug 22, 2024
@mathlib-bors mathlib-bors bot deleted the acmepjz_flat_tosubmodule branch August 22, 2024 16:22
bjoernkjoshanssen pushed a commit that referenced this pull request Sep 9, 2024
bjoernkjoshanssen pushed a commit that referenced this pull request Sep 9, 2024
bjoernkjoshanssen pushed a commit that referenced this pull request Sep 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

easy < 20s of review time. See the lifecycle page for guidelines. t-algebra Algebra (groups, rings, fields, etc)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants