Skip to content

[Merged by Bors] - chore(Logic): reduce use of autoImplicit#14301

Closed
grunweg wants to merge 1 commit intomasterfrom
MR-autoImplicit-logic2
Closed

[Merged by Bors] - chore(Logic): reduce use of autoImplicit#14301
grunweg wants to merge 1 commit intomasterfrom
MR-autoImplicit-logic2

Conversation

@grunweg
Copy link
Copy Markdown
Contributor

@grunweg grunweg commented Jun 30, 2024

The remaining uses are

  • three declarations in Logic/Basic (which need it, somehow)
  • in UnivLE, where removing this yields funky errors related to explicit universe levels
  • in Logic/Equiv/Basic, which is a huge file; fixing it is less trivial

Open in Gitpod

The remaining uses are
- three declarations in Logic/Basic (which need it, somehow)
- in UnivLE, where removing this yields funky errors related to explicit universe levels
- in Logic/Equiv/Basic, which is a huge file; fixing it is less trivial
@grunweg grunweg added awaiting-review easy < 20s of review time. See the lifecycle page for guidelines. t-logic Logic (model theory, etc) labels Jun 30, 2024
@github-actions
Copy link
Copy Markdown

PR summary 4be65ae798

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>

Copy link
Copy Markdown
Member

@eric-wieser eric-wieser left a comment

Choose a reason for hiding this comment

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

bors merge

Thanks!

mathlib-bors bot pushed a commit that referenced this pull request Jun 30, 2024
The remaining uses are
- three declarations in Logic/Basic (which need it, somehow)
- in UnivLE, where removing this yields funky errors related to explicit universe levels
- in Logic/Equiv/Basic, which is a huge file; fixing it is less trivial
@mathlib-bors
Copy link
Copy Markdown
Contributor

mathlib-bors bot commented Jun 30, 2024

Pull request successfully merged into master.

Build succeeded:

@mathlib-bors mathlib-bors bot changed the title chore(Logic): reduce use of autoImplicit [Merged by Bors] - chore(Logic): reduce use of autoImplicit Jun 30, 2024
@mathlib-bors mathlib-bors bot closed this Jun 30, 2024
@mathlib-bors mathlib-bors bot deleted the MR-autoImplicit-logic2 branch June 30, 2024 23:36
@grunweg grunweg restored the MR-autoImplicit-logic2 branch July 1, 2024 09:49
dagurtomas pushed a commit that referenced this pull request Jul 2, 2024
The remaining uses are
- three declarations in Logic/Basic (which need it, somehow)
- in UnivLE, where removing this yields funky errors related to explicit universe levels
- in Logic/Equiv/Basic, which is a huge file; fixing it is less trivial
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-logic Logic (model theory, etc)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants