| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
Sorry, something went wrong.
…owerShell#11193)" This reverts commit 51d2523.
|
Should we add a test for the Get-Help scenario? |
Sorry, something went wrong.
The scenario is a side effect. If we change Get-Help we can lost the new test. I don't know how create more common and reliable test. |
Sorry, something went wrong.
|
Ilya (@iSazonov) not suggesting we change Get-Help itself, but a test for (Get-Help Out-Null).Examples |
Sorry, something went wrong.
|
I will pull the test in follow PR (revert PR is not right place). |
Sorry, something went wrong.
|
Ilya (@iSazonov) Steve Lee (@SteveL-MSFT) Since #11193 was reverted, is there an issue opened to track the original problem that PR was trying to fix? |
Sorry, something went wrong.
|
Dongbo Wang (@daxian-dbw) Original issue was discovered by Steve Lee (@SteveL-MSFT) but not registered so we haven't tracking issue. I do not know whether it is important for Steve Lee (@SteveL-MSFT). |
Sorry, something went wrong.
|
🎉v7.1.0-preview.1 has been released which incorporates this pull request.:tada: Handy links: |
Sorry, something went wrong.
…owerShell#11193)" (PowerShell#11342) This reverts commit ae7ca48.
| Back | FazBrowse Home | New Git URL |
PR Summary
This reverts commit 51d2523. (#11193)
PR Context
Fix #11340
It seems we filter out a null too late.
We need to have more info about a scenario we tried to fix in #11193.
PR Checklist