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

[Merged by Bors] - chore(analysis/normed_space/finite_dimension): golf a proof #13491

Conversation

eric-wieser
Copy link
Member

@eric-wieser eric-wieser commented Apr 17, 2022

These letIs just made this proof convoluted, the instances were not needed.
Without them, we don't even need the import.

Similarly, the classical was the cause of the need for the congr.


Open in Gitpod

These `letI`s just made this proof convoluted, the instances were not needed.
Similarly, the `classical` was the cause of the need for the `congr`.
@eric-wieser eric-wieser added easy < 20s of review time. See the lifecycle page for guidelines. awaiting-review The author would like community review of the PR labels Apr 17, 2022
Copy link
Collaborator

@bryangingechen bryangingechen left a comment

Choose a reason for hiding this comment

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

Thanks!
bors r+

@leanprover-community-bot-assistant leanprover-community-bot-assistant 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 Apr 18, 2022
bors bot pushed a commit that referenced this pull request Apr 18, 2022
These `letI`s just made this proof convoluted, the instances were not needed.
Without them, we don't even need the import.

Similarly, the `classical` was the cause of the need for the `congr`.
@bors
Copy link

bors bot commented Apr 18, 2022

Pull request successfully merged into master.

Build succeeded:

@bors bors bot changed the title chore(analysis/normed_space/finite_dimension): golf a proof [Merged by Bors] - chore(analysis/normed_space/finite_dimension): golf a proof Apr 18, 2022
@bors bors bot closed this Apr 18, 2022
@bors bors bot deleted the eric-wieser/golf-continuous_linear_map.continuous_det branch April 18, 2022 20:47
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
easy < 20s of review time. See the lifecycle page for guidelines. 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