Skip to content
This repository was archived by the owner on Apr 5, 2024. It is now read-only.

Showing extra information about Unity Player/Editor when attaching th… #129

Merged
merged 1 commit into from
Mar 21, 2019

Conversation

rarepops
Copy link
Contributor

Showing the name of the project when trying to attach the debugger
Updating automated tests
Updating packages.json dependencies to newer vUersions.

@@ -88,7 +88,7 @@ static string GetUnityProcesses()
{
var processes = UnityProcessDiscovery.GetAttachableProcesses();

return string.Join("\n", processes.Select(x => x.Name + $" ({x.Id})"));
return string.Join("\n", processes.Select(x => $"{x.Name} ({x.Id})" + $" {x.ProjectName}"));
Copy link
Collaborator

Choose a reason for hiding this comment

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

Just combine these to one interpolated string: $"{x.Name} ({x.Id}) {x.ProjectName}"

Copy link
Collaborator

@miniwolf miniwolf left a comment

Choose a reason for hiding this comment

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

Looks good.

@rarepops rarepops merged commit 08933f5 into master Mar 21, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants