Skip to content

[Merged by Bors] - chore: make Complex.ext only a local ext lemma#9010

Closed
j-loreaux wants to merge 2 commits intomasterfrom
j-loreaux/local-Complex.ext
Closed

[Merged by Bors] - chore: make Complex.ext only a local ext lemma#9010
j-loreaux wants to merge 2 commits intomasterfrom
j-loreaux/local-Complex.ext

Conversation

@j-loreaux
Copy link
Copy Markdown
Contributor

In accordance with this Zulip thread, this remove Complex.ext from the global ext attribute list and only enables it locally in certain files.


I only activated Complex.ext as a local attribute if one of the following conditions were met:

  • The file used Complex.ext multiple times as an ext call and the file was about .
  • The ext call was using multiple lemmas in addition to Complex.ext.

Otherwise, I just substituted apply Complex.ext

Open in Gitpod

@j-loreaux j-loreaux added awaiting-review easy < 20s of review time. See the lifecycle page for guidelines. t-algebra Algebra (groups, rings, fields, etc) labels Dec 12, 2023
@kmill
Copy link
Copy Markdown
Contributor

kmill commented Dec 12, 2023

Looks good to me. Thanks for following up on this -- this ext lemma was annoying me too today.

bors d+

@mathlib-bors
Copy link
Copy Markdown
Contributor

mathlib-bors bot commented Dec 12, 2023

✌️ j-loreaux can now approve this pull request. To approve and merge a pull request, simply reply with bors r+. More detailed instructions are available here.

@github-actions github-actions bot added delegated This pull request has been delegated to the PR author (or occasionally another non-maintainer). and removed awaiting-review labels Dec 12, 2023
@j-loreaux
Copy link
Copy Markdown
Contributor Author

bors merge

@github-actions github-actions bot added the ready-to-merge This PR has been sent to bors. label Dec 12, 2023
mathlib-bors bot pushed a commit that referenced this pull request Dec 12, 2023
In accordance with this [Zulip thread](https://leanprover.zulipchat.com/#narrow/stream/287929-mathlib4/topic/Complex.2Eext/near/393560116), this remove `Complex.ext` from the global `ext` attribute list and only enables it locally in certain files.
@mathlib-bors
Copy link
Copy Markdown
Contributor

mathlib-bors bot commented Dec 12, 2023

Pull request successfully merged into master.

Build succeeded:

@mathlib-bors mathlib-bors bot changed the title chore: make Complex.ext only a local ext lemma [Merged by Bors] - chore: make Complex.ext only a local ext lemma Dec 12, 2023
@mathlib-bors mathlib-bors bot closed this Dec 12, 2023
@mathlib-bors mathlib-bors bot deleted the j-loreaux/local-Complex.ext branch December 12, 2023 23:03
awueth pushed a commit that referenced this pull request Dec 19, 2023
In accordance with this [Zulip thread](https://leanprover.zulipchat.com/#narrow/stream/287929-mathlib4/topic/Complex.2Eext/near/393560116), this remove `Complex.ext` from the global `ext` attribute list and only enables it locally in certain files.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

delegated This pull request has been delegated to the PR author (or occasionally another non-maintainer). easy < 20s of review time. See the lifecycle page for guidelines. 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.

2 participants