Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

remove additional redundant "press any key to exit" #45

Merged
merged 13 commits into from
Oct 14, 2021
Merged

remove additional redundant "press any key to exit" #45

merged 13 commits into from
Oct 14, 2021

Conversation

lovettchris
Copy link
Contributor

@lovettchris lovettchris commented Oct 8, 2021

fix: remove additional redundant "press any key to exit" that was happening only on windows.
fix: remove reference to "Lean 3" in NoDefaultToolchain error message.

The reason for this "where installation happens in a console that may have opened just for this purpose" is no longer valid. The latest vscode extension when you install elan, you now get "press any key to continue" twice, which is odd, so this fixes that making the vscode extension experience consistent across platforms.

@lovettchris lovettchris changed the title make progress output optional, as it looks horrible on windows in powershell terminal in vscode. remove additional redundant "press any key to exit" Oct 9, 2021
src/elan/errors.rs Outdated Show resolved Hide resolved
README.md Show resolved Hide resolved
@Kha Kha merged commit dd07064 into leanprover:master Oct 14, 2021
@Kha
Copy link
Member

Kha commented Oct 14, 2021

A bit late, but I hope this doesn't break/affect the Lean 3 extension's Lean installation? /cc @gebner

@lovettchris
Copy link
Contributor Author

A bit late, but I hope this doesn't break/affect the Lean 3 extension's Lean installation? /cc @gebner

I just tested bootstrapping of lean3 from the vscode lean3 extension and it was able to download and install lean just fine, so looks like it still works.

@gebner
Copy link
Member

gebner commented Oct 14, 2021

I just tested bootstrapping of lean3 from the vscode lean3 extension and it was able to download and install lean just fine, so looks like it still works.

From what I can tell, the change isn't released yet, right?

Either way, the change looks innocent enough and I don't expect it to break anything. To be on the safe side we could post a heads-up on zulip once the release is out.

@lovettchris
Copy link
Contributor Author

lovettchris commented Oct 15, 2021

I just tested bootstrapping of lean3 from the vscode lean3 extension and it was able to download and install lean just fine, so looks like it still works.

From what I can tell, the change isn't released yet, right?

Either way, the change looks innocent enough and I don't expect it to break anything. To be on the safe side we could post a heads-up on zulip once the release is out.

I installed https://github.com/leanprover/vscode-lean changed it to run my locally build elan-init.sh and then invoke my locally built elan-init.exe and it worked fine, including the press enter key to continue prompt.

image

@Kha
Copy link
Member

Kha commented Oct 15, 2021

It's at https://github.com/leanprover/vscode-lean. But I made testing easier by going ahead and releasing the change :) .

@lovettchris lovettchris deleted the clovett/optional_progress_output branch October 15, 2021 06:31
@lovettchris
Copy link
Contributor Author

It's at https://github.com/leanprover/vscode-lean. But I made testing easier by going ahead and releasing the change :) .

Thanks, I tested version 1.1.1 and found another redundant prompt... See #48

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants