-
Notifications
You must be signed in to change notification settings - Fork 38.7k
Closed
Labels
author-verification-requestedIssues potentially verifiable by issue authorIssues potentially verifiable by issue authorbugIssue identified by VS Code Team member as probable bugIssue identified by VS Code Team member as probable buginsiders-releasedPatch has been released in VS Code InsidersPatch has been released in VS Code InsiderstasksTask system issuesTask system issuesverifiedVerification succeededVerification succeeded
Milestone
Description
A task's terminal status becomes "failed" with the hover message "Task has errors" when a problem matcher detected anything, including INFO and WARNING level problems:
vscode/src/vs/workbench/contrib/tasks/browser/taskTerminalStatus.ts
Lines 67 to 71 in eca7184
| if ((event.exitCode === 0) && (terminalData.problemMatcher.numberOfMatches === 0)) { | |
| terminalData.terminal.statusList.add(SUCCEEDED_TASK_STATUS); | |
| } else { | |
| terminalData.terminal.statusList.add(FAILED_TASK_STATUS); | |
| } |
Can we instead either
- purely rely on the task's exit code,
- only consider it failed for ERROR level problems, or
- introduce warning/info status indicators?
Let me know if this makes sense and I can prepare a PR.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
author-verification-requestedIssues potentially verifiable by issue authorIssues potentially verifiable by issue authorbugIssue identified by VS Code Team member as probable bugIssue identified by VS Code Team member as probable buginsiders-releasedPatch has been released in VS Code InsidersPatch has been released in VS Code InsiderstasksTask system issuesTask system issuesverifiedVerification succeededVerification succeeded