Conversation
We no longer strip() the trailing whitespace from Probe output which can result in different behavior than previous versions. Signed-off-by: Sol Jerome <sol.jerome@gmail.com>
|
Is this really an intended change? Btw: And I cannot find a |
|
Do we really want to document this as change? Or do we want to add a |
|
@AlexanderS I believe it was an intended change so that the probes were technically accurate. In other words, I'm guessing someone had a Probe where they wanted the whitespace to stick around. I prefer the old way but I can see where one might run into problems if they care about the trailing whitespace. |
|
Maybe we want to strip the output by default, but add the possibility to flag the Probes that shouldn't be striped somehow? |
We no longer strip() the trailing whitespace from Probe output which can
result in different behavior than previous versions.
Signed-off-by: Sol Jerome sol.jerome@gmail.com