Encoding Parameter for Tee-Object#12135
Encoding Parameter for Tee-Object#12135iSazonov merged 6 commits intoPowerShell:masterfrom Peter-Schneider:TeeObjectEncodingParameter
Conversation
src/Microsoft.PowerShell.Commands.Utility/commands/utility/Tee-Object.cs
Outdated
Show resolved
Hide resolved
src/Microsoft.PowerShell.Commands.Utility/commands/utility/Tee-Object.cs
Outdated
Show resolved
Hide resolved
src/Microsoft.PowerShell.Commands.Utility/commands/utility/Tee-Object.cs
Outdated
Show resolved
Hide resolved
|
Ha, you beat me to it. 😁 |
src/Microsoft.PowerShell.Commands.Utility/commands/utility/Tee-Object.cs
Outdated
Show resolved
Hide resolved
src/Microsoft.PowerShell.Commands.Utility/commands/utility/Tee-Object.cs
Outdated
Show resolved
Hide resolved
|
@vexx32 Thank you anyway.. ;-) |
|
@Peter-Schneider Please add tests. |
test/powershell/Modules/Microsoft.PowerShell.Utility/Tee-Object.Tests.ps1
Outdated
Show resolved
Hide resolved
test/powershell/Modules/Microsoft.PowerShell.Utility/Tee-Object.Tests.ps1
Outdated
Show resolved
Hide resolved
test/powershell/Modules/Microsoft.PowerShell.Utility/Tee-Object.Tests.ps1
Outdated
Show resolved
Hide resolved
test/powershell/Modules/Microsoft.PowerShell.Utility/Tee-Object.Tests.ps1
Outdated
Show resolved
Hide resolved
test/powershell/Modules/Microsoft.PowerShell.Utility/Tee-Object.Tests.ps1
Outdated
Show resolved
Hide resolved
test/powershell/Modules/Microsoft.PowerShell.Utility/Tee-Object.Tests.ps1
Outdated
Show resolved
Hide resolved
test/powershell/Modules/Microsoft.PowerShell.Utility/Tee-Object.Tests.ps1
Outdated
Show resolved
Hide resolved
src/Microsoft.PowerShell.Commands.Utility/commands/utility/Tee-Object.cs
Outdated
Show resolved
Hide resolved
|
@Peter-Schneider Please open new issue in PowerShell-Docs repo for the new parameter and add reference in the PR description. |
|
@SteveL-MSFT @TravisEz13 Could you please look CI failures? It seems it is not the PR related. |
|
@iSazonov The windows failure definitely seem related to this branch. Perhaps a rebase is needed. |
|
@Peter-Schneider Please rebase. You will be switched to new .Net version - 5.0 Preview1, so you should install it. |
test/powershell/Modules/Microsoft.PowerShell.Utility/Tee-Object.Tests.ps1
Outdated
Show resolved
Hide resolved
test/powershell/Modules/Microsoft.PowerShell.Utility/Tee-Object.Tests.ps1
Outdated
Show resolved
Hide resolved
test/powershell/Modules/Microsoft.PowerShell.Utility/Tee-Object.Tests.ps1
Outdated
Show resolved
Hide resolved
test/powershell/Modules/Microsoft.PowerShell.Utility/Tee-Object.Tests.ps1
Outdated
Show resolved
Hide resolved
Co-Authored-By: Ilya <[email protected]>
test/powershell/Modules/Microsoft.PowerShell.Utility/Tee-Object.Tests.ps1
Outdated
Show resolved
Hide resolved
|
This pull request has been automatically marked as Review Needed because it has been there has not been any activity for 7 days. |
|
@TravisEz13 @SteveL-MSFT Could you please review the PR. |
|
This pull request has been automatically marked as Review Needed because it has been there has not been any activity for 7 days. |
|
Reopen to update CIs. |
|
This pull request has been automatically marked as Review Needed because it has been there has not been any activity for 7 days. |
|
@SteveL-MSFT Can we merge? |
|
This pull request has been automatically marked as Review Needed because it has been there has not been any activity for 7 days. |
|
@iSazonov I think this is good to go. thanks. |
|
@Peter-Schneider Thanks for your contribution! |
|
Thanks for merging! 🥳 |
|
🎉 Handy links: |
|
🎉 Handy links: |
PR Summary
Adds an Encoding parameter to the Tee-Object cmdlet (See issue #11104)
PR Context
PR Checklist
.h,.cpp,.cs,.ps1and.psm1files have the correct copyright headerWIP:or[ WIP ]to the beginning of the title (theWIPbot will keep its status check atPendingwhile the prefix is present) and remove the prefix when the PR is ready.