Skip to content
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

bugfix: correct error report when using a shell resource (inverting stdout and stderr) #269

Merged
merged 1 commit into from
Aug 31, 2021

Conversation

dduportal
Copy link
Contributor

@dduportal dduportal commented Aug 31, 2021

bugfix: correct error report when using a shell resource (inverting stdout and stderr)

while writing the documentation for the new shell resource introduced in #264 in updatecli/website#119, I caught this tiny bug (should only be a patch)

Test

To test this pull request, you can run the following commands:

  cp <to_package_directory>
  go test

Additionnal Information

Tradeoff

Potential improvement

…tdout and stderr)

Signed-off-by: Damien Duportal <damien.duportal@gmail.com>
@dduportal dduportal added bug Something isn't working condition Modify condition resource source target Related to updatecli target labels Aug 31, 2021
@dduportal dduportal requested a review from olblak August 31, 2021 11:20
@dduportal dduportal merged commit 89b8ae5 into updatecli:main Aug 31, 2021
@dduportal dduportal deleted the bugfix/shell-report branch August 31, 2021 12:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working condition Modify condition resource source target Related to updatecli target
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants