The Wayback Machine - https://web.archive.org/web/20200915123540/https://github.com/LeetCode-OpenSource/vscode-leetcode/pull/572
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

add re-run last test option #572

Open
wants to merge 4 commits into
base: master
from

Conversation

@nathanesau
Copy link
Contributor

nathanesau commented Jun 12, 2020

re-run last option

Implement request from Issue #570

Nathan Esau added 2 commits Jun 12, 2020
Nathan Esau
Nathan Esau
@@ -42,6 +42,12 @@ export async function testSolution(uri?: vscode.Uri): Promise<void> {
detail: "Test with the written cases in file",
value: ":file",
},
{
label: "$(refresh) Re- run last...",

This comment has been minimized.

@jdneo

jdneo Jun 27, 2020

Member

UI: Use this icon please: $(debug-restart)

reference: https://microsoft.github.io/vscode-codicons/dist/codicon.html

@jdneo
Copy link
Member

jdneo commented Jun 27, 2020

Hey @nathanesau,

Thank you for your contribution, and sorry for the late reply...

I think it could be better if we use the icon debug-restart

@jdneo
Copy link
Member

jdneo commented Jul 23, 2020

@nathanesau, will you still have the interest to update the PR?

--global and others added 2 commits Jul 23, 2020
--global
@nathanesau
Copy link
Contributor Author

nathanesau commented Jul 23, 2020

Sorry for the delay.

I have made the change (commit "use debug-start icon"):

re_run_last

@@ -42,6 +42,12 @@ export async function testSolution(uri?: vscode.Uri): Promise<void> {
detail: "Test with the written cases in file",
value: ":file",
},
{
label: "$(debug-restart) Re-run last...",

This comment has been minimized.

@jdneo

jdneo Jul 24, 2020

Member

One suggestion here: We should only show the re-run option when the lastTest is not empty. Also, this option should put at the top of the list, to make it more convenient for the users.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
2 participants
You can’t perform that action at this time.