Add -PassThru parameter to Set-Clipboard

[ThomasNieto]

PR Summary

Adds a -PassThru parameter to Set-Clipboard.

PS C:\> 'hello' | Set-Clipboard -PassThru
hello
PS C:\> 'hello', 'world' | Set-Clipboard -PassThru
hello
world

PR Context

Resolves #13711

PR Checklist

• PR has a meaningful title
  • Use the present tense and imperative mood when describing your changes
• Summarized changes
• Make sure all .h, .cpp, .cs, .ps1 and .psm1 files have the correct copyright header
• This PR is ready to merge and is not Work in Progress.
  • If the PR is work in progress, please add the prefix WIP: or [ WIP ] to the beginning of the title (the WIP bot will keep its status check at Pending while the prefix is present) and remove the prefix when the PR is ready.
• Breaking changes
  • None
  • OR
  • Experimental feature(s) needed
    • Experimental feature name(s):
• User-facing changes
  • Not Applicable
  • OR
  • Documentation needed
    • Issue filed: Set-Clipboard -PassThru Parameter MicrosoftDocs/PowerShell-Docs#6690
• Testing - New and feature
  • N/A or can only be tested interactively
  • OR
  • Make sure you've added a new test if existing tests do not effectively test the code changed
• Tooling
  • I have considered the user experience from a tooling perspective and don't believe tooling will be impacted.
  • OR
  • I have considered the user experience from a tooling perspective and enumerated concerns in the summary. This may include:
    • Impact on PowerShell Editor Services which is used in the PowerShell extension for VSCode (which runs in a different PS Host).
    • Impact on Completions (both in the console and in editors) - one of PowerShell's most powerful features.
    • Impact on PSScriptAnalyzer (which provides linting & formatting in the editor extensions).
    • Impact on EditorSyntax (which provides syntax highlighting with in VSCode, GitHub, and many other editors).

[ThomasNieto]

@SteveL-MSFT any chance we could squeeze this one in 7.1 since its such a small change? 😄

[ghost]

This pull request has been automatically marked as Review Needed because it has been there has not been any activity for 7 days.
Maintainer, please provide feedback and/or mark it as Waiting on Author

[SteveL-MSFT]

@PowerShell/powershell-committee discussed this and would like to understand a real world scenario usage of this to determine the right behavior or whether we even want to add this

[SteveL-MSFT]

@PowerShell/powershell-committee discussed this again, we agreed that most users expect -PassThru to act like tee in that the intent is to pass input into Set-Clipboard and subsequently pass the original input into something else further down the pipeline (not the transformed result). In addition, it would seem more useful in most cases to get the original object than the stringified version which can be retrieved via Get-Clipboard for that use case. The semantics should not change with -Append. The -Append only affects the clipboard and not what is passed through.

[ghost]

This pull request has been automatically marked as Review Needed because it has been there has not been any activity for 7 days.
Maintainer, please provide feedback and/or mark it as Waiting on Author

[SteveL-MSFT]

I think if a test case can be added for -Append (or perhaps just make it a test case for the single and multiple objects tests, then I'm good with this PR.

[SteveL-MSFT]

I fixed the merge conflict and added -Append test cases

[iSazonov]

@SteveL-MSFT Ready to merge?

[SteveL-MSFT]

Yes, this one is ready. @TravisEz13's concern was addressed, but I'll ask @TravisEz13 to merge since it's assigned to him

[iSazonov]

@ThomasNieto Thanks for you contribution!

[ghost]

🎉v7.2.0-preview.9 has been released which incorporates this pull request.:tada:

Handy links:

• Release Notes

[ghost]

🎉v7.2.0-preview.10 has been released which incorporates this pull request.:tada:

Handy links:

• Release Notes