[vscode] Support optional property TaskPresentationOptions#clear #11133

[jfaltermeier]

What it does

Adds the clear property to the API and the data transfer object.
The functionality was working already as far as I could see, since the flag was available on TaskOutputPresentation already and the conversion code kept this flag in.

Fixes #11133

Contributed on behalf of STMicroelectronics

How to test

I've created an extension (https://github.com/jfaltermeier/vscode-playground/blob/main/tasks-clear/src/extension.ts, https://github.com/jfaltermeier/vscode-playground/releases/download/tasks-clear-0.0.1/tasks-clear-0.0.1.vsix) that adds two commands, "Task with clear" and "Task without clear",
Both will execute a task programmatically that will print text to the terminal. The terminal is cleared depending on the clear flag.

I've also checked with a tasks.json-file:

{
    // See https://go.microsoft.com/fwlink/?LinkId=733558
    // for the documentation about the tasks.json format
    "version": "2.0.0",
    "tasks": [
      {
        "label": "Task with clear",
        "type": "shell",
        "command": "echo 1 && echo 2 && echo 3",
        "presentation": {
          "reveal": "always",
          "panel": "shared",
          "clear": true
        },
        "problemMatcher": []
      },
      {
        "label": "Task without clear",
        "type": "shell",
        "command": "echo 1 && echo 2 && echo 3",
        "presentation": {
          "reveal": "always",
          "panel": "shared",
          "clear": false
        },
        "problemMatcher": []
      },
    ]
  }
  

Review checklist

• As an author, I have thoroughly tested my changes and carefully followed the review guidelines

Reminder for reviewers

• As a reviewer, I agree to behave in accordance with the review guidelines

[msujew]

I can confirm that the changes align the TaskPresentationOptions#clear property to vscode. The functionality of the property continues to work as expected 👍

[msujew]

@jfaltermeier There are merge conflicts, I assume related to the changelog entry. Do you mind resolving those, I'll then try to merge this PR in a timely manner.

[vince-fugnitto]

LGTM 👍

[JonasHelming]

@jfaltermeier There are merge conflicts, I assume related to the changelog entry. Do you mind resolving those, I'll then try to merge this PR in a timely manner.

Any reason why @jfaltermeier should not merge himselve after resolving the conflict? @msujew ?

[msujew]

@JonasHelming I forgot that he's a member of the team. Kind of hard keeping up with all you EclipseSource folks in the project ;)