| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
Sorry, something went wrong.
update resource namespaces to match
suppress newline for show-command output as it doesn't work with psreadline
|
Marking it WIP as I work through the Codacy and Codefactor issues |
Sorry, something went wrong.
|
I still think this should be an external module 😞 |
Sorry, something went wrong.
|
Joel Bennett (@Jaykul) did you read the PR description above? |
Sorry, something went wrong.
There was a problem hiding this comment.
Reviewed commit by commit.
Sorry, something went wrong.
|
Sean Wheeler (@sdwheeler) does Show-Command also need a new fwlink? |
Sorry, something went wrong.
|
Steve Lee (@SteveL-MSFT) Show-Command https://go.microsoft.com/fwlink/?linkid=2109589
|
Sorry, something went wrong.
|
Steve Lee (@SteveL-MSFT) please resolve merge conflict. Also, open a PR in the PowerShell-Native repo to add the new assembly in the trusted assemblies list:
|
Sorry, something went wrong.
|
Aditya Patwardhan (@adityapatwardhan) does it make sense to add it to trusted assemblies since it can't be used remotely? |
Sorry, something went wrong.
|
Steve Lee (@SteveL-MSFT) merging this PR. I can confirm with Paul Higinbotham (@PaulHigin) if the change in trusted assemblies must be made. It does not block this PR. |
Sorry, something went wrong.
|
🎉v7.0.0-preview.6 has been released which incorporates this pull request.:tada: Handy links: |
Sorry, something went wrong.
|
Get-Help -ShowWindow only shows the first paragraph of text for each parameter. This is a bug from Windows PowerShell that seems to have been carried over |
Sorry, something went wrong.
|
RichardSiddaway No plans exist to fix the old code. All new graphical cmdlets will be in https://github.com/PowerShell/GraphicalTools Also MSFT team has a plan to rewrite Help module.
|
Sorry, something went wrong.
|
RichardSiddaway, Ilya (@iSazonov) is correct, this is simply a straight port with no plans to invest in these tools. They are added for parity with Windows PowerShell. |
Sorry, something went wrong.
|
That's a shame that a long standing bug has been allowed to propagate into v7 |
Sorry, something went wrong.
|
RichardSiddaway We are not happy too. I hope there are people who will port the cmdlets. Welcome to these projects. |
Sorry, something went wrong.
Sorry, something went wrong.
…, and `Get-Help -ShowWindow` (PowerShell#10899)
| Back | FazBrowse Home | New Git URL |
PR Summary
Add back GraphicalHost assembly code to enable Out-GridView, Show-Command, and Get-Help -ShowWindow on Windows only.
Some of the original code was changed for updated copyright and some style issues. Because of a difference in the Windows build system vs dotnet msbuild on embedding resources, had to change resgen to work with what XAML expects. I would focus the code review on source code that was already in this repo that had to be changed to make this work. The original WPF code is largely unchanged except where needed to get it to compile.
No automated tests so I did some manual validation for all three commands.
In the future, we'll probably want to split this out as a separate module, but currently part of the code is in SMA.dll and the rest is in GraphicalHost.dll. Removing the SMA.dll code is not trivial as we need to expose new public APIs for the module in how the cmdlet talks to the WPF window. Also the cmdlet is part of Utility module which is a breaking change in itself.
Note the source folder is the namespace name as resgen uses that to produce the path to the resources and the code expects that path.
PR Context
Fix #10599
Fix #9778
PR Checklist