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

Add 'coreclr' to keywords or other searchable #5973

Closed
timheuer opened this issue Jul 31, 2023 · 9 comments · Fixed by #6071
Closed

Add 'coreclr' to keywords or other searchable #5973

timheuer opened this issue Jul 31, 2023 · 9 comments · Fixed by #6071

Comments

@timheuer
Copy link
Member

When someone has an environment that doesn't have the extension installed and they attempt to debug, a dialog will show:
image

VSCode creates a link install coreclr extension which is just a link to a pre-configured search that is category:"debuggers" coreclr which will return nothing for C#.

image

If we added the keyword and/or ensured it was in something that was indexed this would provide a better experience in this situation.

@dibarbet dibarbet added this to the Next milestone Aug 4, 2023
@dibarbet
Copy link
Member

dibarbet commented Aug 4, 2023

@timheuer if you were thinking of adding 'coreclr' as a keyword here - https://github.com/dotnet/vscode-csharp/blob/main/package.json#L25 I don't believe we can. We're already at the limit of 5

image

I don't know of other places where we can configure the search (but my knowledge here is limited).

@timheuer
Copy link
Member Author

timheuer commented Aug 4, 2023

@dibarbet do we know the history of "multi-root ready" keyword? that doesn't seem terribly valuable and can replace? Ironically if you look at marketplace listing it has more than 5.

@dibarbet
Copy link
Member

dibarbet commented Aug 4, 2023

I don't know the history myself, but this is the PR that added it - #1841

Maybe @DustinCampbell remembers?

I think multi-root ready indicates that you can open multiple folders. Both O# / Roslyn only open 1 sln at a time though (unless load projects on demand works for multi-workspace in O#? Not sure).

@timheuer
Copy link
Member Author

timheuer commented Aug 4, 2023

I wonder if that is a hard-coded UI search thing in VS Code. The one for 'coreclr' is -- when a debugger fails, VSCode basically has a link to pre-search and it's not a great experience. I'd favor using one of our slots for this.

@DustinCampbell
Copy link
Member

Yes, multi-root workspaces allow multiple folders to be opened at once. We made O# work with multi-root workspaces back in the day, but the support was pretty minimal. Instead of removing that, I would imagine that we should try to understand why there's a limit of 5. That seems unnecessarily restrictive.

@timheuer
Copy link
Member Author

timheuer commented Aug 8, 2023

We can do that @DustinCampbell but I'm also just trying to understand if that's even a valuable keyword. 'coreclr' explicitly shows up (think fwlink as the dialog pre-calculates a search) and is failing customers. 'multi-root' seems odd unless similar.

@timheuer
Copy link
Member Author

timheuer commented Aug 8, 2023

Fyi I'm told that the validation of only 5 keywords does not check. In fact I just did it myself: https://marketplace.visualstudio.com/items?itemName=TimHeuer.base-ext-sample

@dibarbet
Copy link
Member

dibarbet commented Aug 8, 2023

Fyi I'm told that the validation of only 5 keywords does not check. In fact I just did it myself: https://marketplace.visualstudio.com/items?itemName=TimHeuer.base-ext-sample

Hmm, might as well try it then! #6071

@DustinCampbell
Copy link
Member

We can do that @DustinCampbell but I'm also just trying to understand if that's even a valuable keyword. 'coreclr' explicitly shows up (think fwlink as the dialog pre-calculates a search) and is failing customers. 'multi-root' seems odd unless similar.

I have no idea if it's still valuable. It was requested that we add that keyword here: https://github.com/Microsoft/vscode/wiki/Adopting-Multi-Root-Workspace-APIs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants