Josh Chorlton

Results 27 comments of Josh Chorlton

@dcantah what's the path to getting this merged?

my editor also removed some unnecessary whitespace

I actually looked further into this, and found that this code is just disabling a warning for when TLS validation fails. It doesn't disable TLS validation. However, this code *does...

I've hit this as well, when a `cp` fails for any reason and it silently fails to stage out. Even worse if the output is optional, because nextflow can't differentiate...

Thanks for the response. I agree that things get challenging with multi-threading. One example of how another testing system handles this is bazel's `--test_output=streamed`: https://bazel.build/reference/command-line-reference#flag--test_output From the docs: > 'streamed'...

Thanks for accepting! I'll get a PR going when I get the chance and we can try to find an elegant implementation.

I spent some time investigating this. The issue seemingly stems from here: https://github.com/MultiQC/MultiQC/blob/7a75a7eb482a36c8fe99363da37cc84362cbb147/multiqc/core/special_case_modules/custom_content.py#L183 For the same ID, because the data is a list, it overwrites the data when it processes...