Skip to content

Conversation

@JoeRobich
Copy link
Member

Removed some docs for no longer being relevant. Updated links as appropriate. Hopefully we could remove the Wiki section following this change.

CC: @gregg-miskelly for sign-off on debugger docs

@JoeRobich JoeRobich requested a review from a team as a code owner August 27, 2025 00:02
@gregg-miskelly
Copy link
Contributor

Is the motivation for doing this that we want to make it easier for the communicate to contribute to documentation?

We can certainly do this, but we will need to work in stages as we have a bunch of aka.ms links that point at the Wiki. So, after this PR merges, we will need to edit the aka.ms links, and then you can delete the Wiki.

@gregg-miskelly
Copy link
Contributor

Looks like there is one place in the code where we reference the wiki directly (not through an aka.ms link) -- https://github.com/search?q=repo%3Adotnet%2Fvscode-csharp+remoteDebugInfoUrl&type=code

I think that code path is from back when this extension didn't support ARM64. I can submit a PR to delete that code.

@gregg-miskelly
Copy link
Contributor

gregg-miskelly commented Aug 28, 2025

There are a few places in 'debugger.md' that have links to the wiki that need to be updated. Search for https://github.com/dotnet/vscode-csharp/wiki.

@gregg-miskelly
Copy link
Contributor

gregg-miskelly commented Aug 28, 2025

Looks like there is one place in the code where we reference the wiki directly (not through an aka.ms link) -- https://github.com/search?q=repo%3Adotnet%2Fvscode-csharp+remoteDebugInfoUrl&type=code

I think that code path is from back when this extension didn't support ARM64. I can submit a PR to delete that code.

Fixed with: #8574

@JoeRobich
Copy link
Member Author

Is the motivation for doing this that we want to make it easier for the communicate to contribute to documentation?

Yes, also it has been my experience that it is generally easier for us to keep it up-to-date and accurate, since we can ask for updates in PRs that change documented behavior. I certainly don't plan to remove them until I get your OK to do so.

@JoeRobich JoeRobich merged commit fd38eee into main Aug 29, 2025
32 checks passed
@JoeRobich JoeRobich deleted the dev/jorobich/move-wiki branch August 29, 2025 20:48
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