Skip to content

Initial CodeTours for contributing to VS Code and Visual Studio extensions #131

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

Open
wants to merge 9 commits into
base: main
Choose a base branch
from

Conversation

TMA-2
Copy link

@TMA-2 TMA-2 commented Mar 16, 2025

For issue #129 - These are more or less placeholder CodeTours consisting of the content on the Wiki for building the extensions, but associated with files and directories where applicable. Also, a VSCode workspace, which may not be totally necessary, but having some VSCode configurations to assist in getting a similar environment set up would certainly help me.

@adamdriscoll
Copy link
Member

This is looking good to me! Let me know if you feel like this is ready to merge.

@TMA-2
Copy link
Author

TMA-2 commented Apr 16, 2025

This is looking good to me! Let me know if you feel like this is ready to merge.

Thanks very much — I'm relatively new to using git so I wasn't sure I got it right (as evidenced by the many needless back-and-forth commits beforehand...).

I think one thing might need updating, though:

  • The Visual Studio tour, step 11: PowerShellTools.MSBuild might need to actually reference PowerShellToolsPro.MsBuild? I could be mistaken, you would you know your own repo but wanted to verify.

Also, I generated some PlatyPS help files for most of the PowerShellProTools.VSCode module as they seem to lack any help content. Besides finishing those and generating MAML help, would it be worth adding in-line XMLDoc documentation to the cmdlet files, i.e. SetStatusBarMessageCommand.cs?

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.

2 participants