-
Notifications
You must be signed in to change notification settings - Fork 18
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
Write-*Log CmdLets do not support pipeline input #19
Comments
mamidenn
added a commit
to mamidenn/PoShLog
that referenced
this issue
Jun 27, 2021
mamidenn
added a commit
to mamidenn/PoShLog
that referenced
this issue
Jun 27, 2021
hi again Martin, you rock 👍 thanks for fixing this I have some unfinished changes in PoShLog, so I will release your fix when I'm done. |
TomasBouda
pushed a commit
that referenced
this issue
Jul 2, 2021
Hi, I've relesed 2.2.0-preview1 to test some changes. You can verify your bugfixes there👍 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
I would like to be able to pipe the output of CmdLets or called programs into
Write-*Log
. Currently, only the last element of the input gets actually logged.Expected:
Actual:
This is due to the CmdLets using
ValueFromPipeline
for the$MessageTemplate
parameter but not wrapping the execution body in aProcess
block. Basically, all that needs to be done is replace thiswith this in all the CmdLets:
I can open a PR if you are interested in this.
Also big props for this nice project 👍.
The text was updated successfully, but these errors were encountered: