Skip to content

[Merged by Bors] - feat: generalize universes in preservesFiniteProductsOfPreservesBinaryAndTerminal#16178

Closed
callesonne wants to merge 2 commits intomasterfrom
calle/generalize-type-universe
Closed

[Merged by Bors] - feat: generalize universes in preservesFiniteProductsOfPreservesBinaryAndTerminal#16178
callesonne wants to merge 2 commits intomasterfrom
calle/generalize-type-universe

Conversation

@callesonne
Copy link
Copy Markdown
Collaborator

@callesonne callesonne commented Aug 27, 2024


I noticed some theorems that hold with more general universe parameters than given. Since this is just a generalization of a lemma, I assume there is no harm in doing this?

Open in Gitpod

@callesonne callesonne added the easy < 20s of review time. See the lifecycle page for guidelines. label Aug 27, 2024
@github-actions
Copy link
Copy Markdown

PR summary a79abb66ec

Import changes for modified files

No significant changes to the import graph

Import changes for all files
Files Import difference

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/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.

@callesonne callesonne added the awaiting-CI This PR does not pass CI yet. This label is automatically removed once it does. label Aug 27, 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 Aug 27, 2024
@dagurtomas
Copy link
Copy Markdown
Contributor

In this direction, this is an obvious improvement. Thanks!

maintainer merge

@github-actions
Copy link
Copy Markdown

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

@github-actions github-actions bot added the maintainer-merge A reviewer has approved the changed; awaiting maintainer approval. label Aug 27, 2024
@dagurtomas dagurtomas changed the title feat: generalize some universe arguments feat: preseves terminal and binary products implies preserves finite product indexed by a finite type in an arbitrary universe Aug 27, 2024
@dagurtomas dagurtomas changed the title feat: preseves terminal and binary products implies preserves finite product indexed by a finite type in an arbitrary universe feat: generalize universes in preservesFiniteProductsOfPreservesBinaryAndTerminal Aug 27, 2024
@mattrobball
Copy link
Copy Markdown
Contributor

bors merge

@github-actions github-actions bot added the ready-to-merge This PR has been sent to bors. label Aug 27, 2024
mathlib-bors bot pushed a commit that referenced this pull request Aug 27, 2024
@mathlib-bors
Copy link
Copy Markdown
Contributor

mathlib-bors bot commented Aug 27, 2024

Pull request successfully merged into master.

Build succeeded:

@mathlib-bors mathlib-bors bot changed the title feat: generalize universes in preservesFiniteProductsOfPreservesBinaryAndTerminal [Merged by Bors] - feat: generalize universes in preservesFiniteProductsOfPreservesBinaryAndTerminal Aug 27, 2024
@mathlib-bors mathlib-bors bot closed this Aug 27, 2024
@mathlib-bors mathlib-bors bot deleted the calle/generalize-type-universe branch August 27, 2024 12:47
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. maintainer-merge A reviewer has approved the changed; awaiting maintainer approval. ready-to-merge This PR has been sent to bors.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants