Skip to content

chore: reduce imports#533

Merged
joehendrix merged 3 commits intomainfrom
shake_list_lemmas
Jan 17, 2024
Merged

chore: reduce imports#533
joehendrix merged 3 commits intomainfrom
shake_list_lemmas

Conversation

@kim-em
Copy link
Copy Markdown
Collaborator

@kim-em kim-em commented Jan 16, 2024

Reduce imports.

@kim-em
Copy link
Copy Markdown
Collaborator Author

kim-em commented Jan 16, 2024

@digama0, do you know why lake exe shake Std (run from Mathlib) doesn't report this?

@kim-em kim-em added the awaiting-review This PR is ready for review; the author thinks it is ready to be merged. label Jan 16, 2024
@digama0
Copy link
Copy Markdown
Member

digama0 commented Jan 16, 2024

Uh oh, there was a bug which after fixing is generating loads more shake reports...

@kim-em
Copy link
Copy Markdown
Collaborator Author

kim-em commented Jan 16, 2024

Good news! :-)

@digama0
Copy link
Copy Markdown
Member

digama0 commented Jan 16, 2024

I pushed the results of the shake report to this branch. I'll make a PR separately to mathlib with the noshake.json changes for std, such that running lake exe shake Std works as expected. (This is a bit of a layering violation, we should probably upstream shake if we want to keep this working.)

@joehendrix joehendrix added will-merge-soon PR will be merged soon. Any concerns should be raised quickly. and removed awaiting-review This PR is ready for review; the author thinks it is ready to be merged. labels Jan 17, 2024
@joehendrix joehendrix merged commit ee4a0f1 into main Jan 17, 2024
@joehendrix joehendrix deleted the shake_list_lemmas branch January 17, 2024 20:06
mathlib-bors bot pushed a commit to leanprover-community/mathlib4 that referenced this pull request Jan 23, 2024
- [ ] depends on: #9830

This is the `noshake.json` file corresponding to the edits in leanprover-community/batteries#533. There was a small bug in `shake` making it much more conservative than it needed to be, so there is a big mathlib PR #9830, on which this PR depends because CI will not pass without it.



Co-authored-by: Mario Carneiro <di.gama@gmail.com>
Co-authored-by: Scott Morrison <scott.morrison@gmail.com>
fgdorais pushed a commit to fgdorais/batteries that referenced this pull request Feb 18, 2024
Co-authored-by: Mario Carneiro <di.gama@gmail.com>
fgdorais pushed a commit to fgdorais/batteries that referenced this pull request Feb 18, 2024
Co-authored-by: Mario Carneiro <di.gama@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

will-merge-soon PR will be merged soon. Any concerns should be raised quickly.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants