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

Clear doesn't work for extension contributed tasks #62474

Closed
dbaeumer opened this issue Nov 2, 2018 · 2 comments
Closed

Clear doesn't work for extension contributed tasks #62474

dbaeumer opened this issue Nov 2, 2018 · 2 comments
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug candidate Issue identified as probable candidate for fixing in the next release tasks Task system issues verification-found Issue verification failed
Milestone

Comments

@dbaeumer
Copy link
Member

dbaeumer commented Nov 2, 2018

To reproduce:

package.json

	"scripts": {
		"echo": "node -e console.log('Hello')"
	},

tasks.json

		{
			"type": "npm",
			"script": "echo",
			"presentation": {
				"echo": true,
				"reveal": "always",
				"focus": false,
				"panel": "shared",
				"showReuseMessage": true,
				"clear": true
			},
			"problemMatcher": []
		}

Observe: terminal is not cleared.

To make this work we need to add the clear here: https://github.com/Microsoft/vscode/blob/master/src/vs/workbench/parts/tasks/node/taskConfiguration.ts#L750

Sorry for not catching this when reviewing.

@dbaeumer dbaeumer added tasks Task system issues bug Issue identified by VS Code Team member as probable bug labels Nov 2, 2018
Areax added a commit to Areax/vscode that referenced this issue Nov 2, 2018
@alexr00 alexr00 added the candidate Issue identified as probable candidate for fixing in the next release label Nov 5, 2018
@isidorn isidorn added this to the October 2018 milestone Nov 5, 2018
@alexr00 alexr00 closed this as completed in 90fe5c8 Nov 5, 2018
@alexr00
Copy link
Member

alexr00 commented Nov 5, 2018

Reopening until this is merged into release.

@alexr00 alexr00 reopened this Nov 5, 2018
@alexr00 alexr00 closed this as completed Nov 6, 2018
@RMacfarlane RMacfarlane added verified Verification succeeded verification-found Issue verification failed and removed verified Verification succeeded labels Nov 9, 2018
@RMacfarlane
Copy link
Contributor

Following the steps above, I'm still seeing the task output left in the terminal with the final message "Terminal will be reused by tasks, press any key to close it.". I assume this is what should be cleared.

VSCode version:

Version: 1.29.0-insider
Commit: 5e48194
Date: 2018-11-09T14:55:36.370Z

@vscodebot vscodebot bot locked and limited conversation to collaborators Dec 21, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Issue identified by VS Code Team member as probable bug candidate Issue identified as probable candidate for fixing in the next release tasks Task system issues verification-found Issue verification failed
Projects
None yet
Development

No branches or pull requests

4 participants