Skip to content

Update Issue Template #610

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

Merged
merged 1 commit into from
Mar 23, 2017
Merged

Update Issue Template #610

merged 1 commit into from
Mar 23, 2017

Conversation

mattmcnabb
Copy link
Contributor

Now that we have $PSEditor in the terminal, we can run a command to return the PowerShell extension version for issues. Look like a good idea?

@msftclas
Copy link

@mattmcnabb,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by Microsoft. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

@rkeithhill
Copy link
Contributor

Hehe, take a look at PowerShell/PowerShellEditorServices#404 After a bit more thought, I'm beginning to think that a VSCode command is the way to go. PowerShell: Copy Version Info to Clipboard or something like that. Also, David just merged in a PR from Trevor that addresses this as well #607.

@daviwil
Copy link
Contributor

daviwil commented Mar 23, 2017

I think Keith's approach definitely makes the user experience easier, but I'll go ahead and accept Matt's PR because it's useful information :)

@daviwil daviwil merged commit db934e8 into PowerShell:develop Mar 23, 2017
@mattmcnabb
Copy link
Contributor Author

@rkeithhill Yeah I think I was piggybacking on Trevor's PR but added the $PSEditor command. I like the idea of a simple command that gathers all the info as well, could probably work on that.

Also, does it make sense to try to gather the OS name and version as well, or should we just leave that up to the user? Just can't think of a simple way to grab the OS Edition other than WMI, but version should be relatively easy.

@rkeithhill
Copy link
Contributor

@mattmcnabb Look at that PSES PR (404) I linked to above. That command does grab OS info as well.

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.

4 participants