Skip to content
This repository was archived by the owner on Jul 24, 2024. It is now read-only.

[Merged by Bors] - feat(data/support): add function.mul_support#6791

Closed
urkud wants to merge 1 commit intomasterfrom
mul-support
Closed

[Merged by Bors] - feat(data/support): add function.mul_support#6791
urkud wants to merge 1 commit intomasterfrom
mul-support

Conversation

@urkud
Copy link
Copy Markdown
Member

@urkud urkud commented Mar 21, 2021

This will help us add finprod in #6355


Open in Gitpod

This will help us add `finprod` in #6355
Copy link
Copy Markdown
Collaborator

@kim-em kim-em left a comment

Choose a reason for hiding this comment

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

LGTM

@eric-wieser
Copy link
Copy Markdown
Member

bors merge

@github-actions github-actions bot added ready-to-merge All that is left is for bors to build and merge this PR. (Remember you need to say `bors r+`.) and removed awaiting-review The author would like community review of the PR labels Mar 22, 2021
bors bot pushed a commit that referenced this pull request Mar 22, 2021
This will help us add `finprod` in #6355
@bors
Copy link
Copy Markdown

bors bot commented Mar 22, 2021

Build failed (retrying...):

bors bot pushed a commit that referenced this pull request Mar 22, 2021
This will help us add `finprod` in #6355
@bors
Copy link
Copy Markdown

bors bot commented Mar 22, 2021

Pull request successfully merged into master.

Build succeeded:

@bors bors bot changed the title feat(data/support): add function.mul_support [Merged by Bors] - feat(data/support): add function.mul_support Mar 22, 2021
@bors bors bot closed this Mar 22, 2021
@bors bors bot deleted the mul-support branch March 22, 2021 23:38
b-mehta pushed a commit that referenced this pull request Apr 2, 2021
This will help us add `finprod` in #6355
YaelDillies added a commit to leanprover-community/mathlib4 that referenced this pull request Dec 9, 2023
`Function.support` is a very basic definition. Nevertheless, it is a pretty heavy import because it imports most objects a `support` lemma can be written about.

This PR reverses the dependencies between those objects and `Function.support`, so that the latter can become a much more lightweight import. Only one import could not easily be reversed, namely the one to `Data.Set.Finite`, so I created a new file instead.

I credit:
* Yury for leanprover-community/mathlib3#6791
* Oliver for leanprover-community/mathlib3#7439
mathlib-bors bot pushed a commit to leanprover-community/mathlib4 that referenced this pull request Dec 13, 2023
`Function.support` is a very basic definition. Nevertheless, it is a pretty heavy import because it imports most objects a `support` lemma can be written about.

This PR reverses the dependencies between those objects and `Function.support`, so that the latter can become a much more lightweight import.

Only two import could not easily be reversed, namely the ones to `Data.Set.Finite` and `Order.ConditionallyCompleteLattice.Basic`, so I created two new files instead.

I credit:
* Yury for leanprover-community/mathlib3#6791
* Oliver for leanprover-community/mathlib3#7439
awueth pushed a commit to leanprover-community/mathlib4 that referenced this pull request Dec 19, 2023
`Function.support` is a very basic definition. Nevertheless, it is a pretty heavy import because it imports most objects a `support` lemma can be written about.

This PR reverses the dependencies between those objects and `Function.support`, so that the latter can become a much more lightweight import.

Only two import could not easily be reversed, namely the ones to `Data.Set.Finite` and `Order.ConditionallyCompleteLattice.Basic`, so I created two new files instead.

I credit:
* Yury for leanprover-community/mathlib3#6791
* Oliver for leanprover-community/mathlib3#7439
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

ready-to-merge All that is left is for bors to build and merge this PR. (Remember you need to say `bors r+`.)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants