Jesse Glick
Jesse Glick
I am not a maintainer of this plugin (and do not even know much about it) so I am not sure why you are mentioning me.
Without knowing anything about Bats internals or exotic options, I find it hard to understand how this could require deep changes. The text I want to see is already being...
Maybe you are passing some options which block streaming output even to the file? I tested the sample test script given in the issue description with this workaround using Bats...
Maybe that breaks the workaround just by creating multiple out files? The following seems to print output at least for a single test and in a terminal: ```diff diff --git...
Thanks for finding this! Unclear to me that the modified test is still asserting what aa4901c5f85e62a3617222bda42cf00deec4de2f intended, whatever that was. The test should perhaps be rewritten to use some more...
Should be done via BOM.
cleanup request: replace all the `readYaml()` occurences by an agent command (`yq`, `sed`, whatever)
From #30. Need to replace at least https://github.com/jenkins-infra/pipeline-library/blob/5288902cac5871e540ae9146e15e47cb47e6d757/vars/runATH.groovy#L41-L64. Really this whole file should have been an external script. (It _does_ use `parallel`, but apparently only within a single `node`, which...
cleanup request: replace all the `readYaml()` occurences by an agent command (`yq`, `sed`, whatever)
Sounds like overkill. More productive to include more tests in the smoke category. From my PoV the priority is to inline this library function, stripping out unused logic branches in...
cleanup request: replace all the `readYaml()` occurences by an agent command (`yq`, `sed`, whatever)
> why this was implemented here There was a whole plan which was never fully specified, much less implemented, and so we got some half-baked pieces introduced in a few...
cleanup request: replace all the `readYaml()` occurences by an agent command (`yq`, `sed`, whatever)
Needs https://github.com/jenkinsci/jenkins/pull/7438 then https://github.com/jenkins-infra/pipeline-library/pull/525.