Fix buffering issues with xcpretty. #123
Conversation
| STDOUT.print(formatted_text + formatter.optional_newline) unless formatted_text.empty? | ||
| unless formatted_text.empty? | ||
| STDOUT.print(formatted_text + formatter.optional_newline) | ||
| STDOUT.flush() |
There was a problem hiding this comment.
Do not use parentheses for method calls with no arguments.
|
@mtauraso thanks for the contribution! Do you mind posting a screenshot before/after, just documentation-wise? |
|
@mtauraso /ping I've tried to run with or without .flush and can't see any difference. |
|
We always have this buffering issues when run from Jenkins with tee. It would take about 30 seconds to get some output. You can test that by running a job from Jenkins connected through ssh to a remote Slave by running the command below: I tried this patch, and it helps. Now all looks like in real time. @supermarin please merge this as it would fix this buffering issues. |
|
@xfreebird oh i see, it totally makes sense. thanks! |
Fix buffering issues with xcpretty.
This makes the output update more smoothly :)