-
Notifications
You must be signed in to change notification settings - Fork 335
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
Fix 1255-possible-regression-appendenv-in-powershell-fails-if-env-variable-does-not-exist-already #1285
Fix 1255-possible-regression-appendenv-in-powershell-fails-if-env-variable-does-not-exist-already #1285
Conversation
Fixes #1255 |
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
What needs to be done to get this merged? |
At least two approvals for the TSC members. I already internally requested a review on this. |
I would value a comment explaining why the command is set to silently ignore errors. It's one of those things I could see getting removed in the future if no one remembers why it was added. |
681b242
to
4accb0a
Compare
Signed-off-by: Thorsten Kaufmann <[email protected]>
Signed-off-by: Thorsten Kaufmann <[email protected]>
Signed-off-by: Thorsten Kaufmann <[email protected]>
f6d2cf4
to
e3f0020
Compare
…rshell-fails-if-env-variable-does-not-exist-already Signed-off-by: Thorsten Kaufmann <[email protected]>
There is a regression that was introduced by #1176
The nested
Get-ChildItem
call fails with an error if the Variable does not exist. Old behaviour restored without reverting the fix by adding-ErrorAction SilentlyContinue