Skip to content

Conversation

@rajkumar42
Copy link
Contributor

This feature will light up when the required vscode features are available in stable release.

This feature will light up when the required vscode features are available in stable release.
package.json Outdated
"onCommand:o.showOutput",
"onCommand:dotnet.restore",
"onCommand:csharp.downloadDebugger",
"onCommand:debugger.pickProcess",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

'debugger.pickProcess' sounds likely to clash with other extensions. Should this be 'csharp.debuggerPickProcess' so something like that?

@gregg-miskelly
Copy link
Contributor

gregg-miskelly commented Jun 6, 2016

Do we want this in for the next release or not? Rajkumar and I talked about it in person: our opinion - this feels like it isn't 100% baked, so we will put it into dev after we next merge dev->master (hopefully very soon).

- Changing the command to csharp.listProcess.
- Sorting the process, such that dotnet process are shown at the top.
@gregg-miskelly gregg-miskelly merged commit c2b5c1b into dotnet:dev Jun 8, 2016
gregg-miskelly pushed a commit to gregg-miskelly/vscode-csharp that referenced this pull request Jun 16, 2016
* Adding attach process picker.

This feature will light up when the required vscode features are available in stable release.

* Incorporating code review comments.

- Changing the command to csharp.listProcess.
- Sorting the process, such that dotnet process are shown at the top.
gregg-miskelly added a commit that referenced this pull request Jun 17, 2016
* Adding attach process picker.

This feature will light up when the required vscode features are available in stable release.

* Incorporating code review comments.

- Changing the command to csharp.listProcess.
- Sorting the process, such that dotnet process are shown at the top.
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