-
-
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
Broken with none POSIX shell #9
Comments
I just played around with |
Would be great if you find a solution to this, I have no experience with nushell. If you run lollypops with the One possible solution would be to wrap the commands in a
But maybe there is a simpler way of archiving this. |
I spent some time looking into it. The "issue" (it's actually expected behavior) is indeed linked to Would that be a satisfactory solution, or is it a bit too hacky? I can understand if you'd rather avoid this solution for what's essentially a niche use case (I doubt that many people use a non-POSIX shell). |
You can take a look at this commit to see the changes it would create. Nothing suspential but it enforces the use of |
I'm okay with wrapping it in |
It sadly doesn't keep the exit code of the desired command but rather the exit code of The only exception I found so far was with the As far as other implications, configuration of the default remote shell won't be taken into account (unless the default shell is |
I'm using nushell as the default shell on my system and when trying to run an action, let's say
deploy-secrets
, I get the following error:The
mkdir
implementation of nushell doesn't have the-p
option has it creates missing folder by default. I'm not familiar withgo-task
but would there be a way to force the use a POSIX compliant shell (saysh
) to avoid such conflicts ?The text was updated successfully, but these errors were encountered: