-
Notifications
You must be signed in to change notification settings - Fork 125
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
read "go test" output from stdin #414
Comments
Work in progress, feedback on command line flags welcome: #415 |
Hello! I believe it is possible to run from stdin. The readme has this example:
I've generally suggested against running using stdin for a couple reasons:
That said, I think build failures were added to The status code I think is still a risk, so I wouldn't want to suggest this approach in general, but for a sufficient complex setup (as I imagine is the case for kubernetes) that's probably a fine tradeoff. I've been very busy with life outside of github lately, but I will find some time to review your PR. Thank you! |
Indeed, these are the two drawbacks. The documentation in #415 addresses both. For reference, this is the shell code which I found too hard to convert into a generated script: A bash script which currently contains a bash script would have to be turned into a bash script which generates a bash script which contains a bash script... |
I'm checking whether we want to go the |
In Kubernetes, we run fairly complex commands to execute our unit tests. gotestsum is then used for post-processing with
--raw-command cat <filename>
.It might be possible to turn those complex commands into a script (I'm trying...) but the shell quoting is nasty.
Would you accept a patch that makes
gotestsum
read input from stdin?The text was updated successfully, but these errors were encountered: