-
-
Notifications
You must be signed in to change notification settings - Fork 468
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 exit #1797
Fix exit #1797
Conversation
/fyi @asears |
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.
No go unfortunately.
We should probably also add some tests for this as it's hard to remember all scenarios, 🙂
@fflaten should be better. |
Co-authored-by: Frode Flaten <3436158+fflaten@users.noreply.github.com>
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.
@nohwnd Looks good - just need to reduce the sample exitcode for Unix-compatibility
Btw. when would you use |
Co-authored-by: Frode Flaten <3436158+fflaten@users.noreply.github.com>
Co-authored-by: Frode Flaten <3436158+fflaten@users.noreply.github.com>
Co-authored-by: Frode Flaten <3436158+fflaten@users.noreply.github.com>
Co-authored-by: Frode Flaten <3436158+fflaten@users.noreply.github.com>
IDK actually, I removed it to avoid code that is not needed. I could not come up with any way to make it propagate from powershell session into the calling process. |
Fix #1795