For internal use only: add a -text-only flag to src batch [apply|preview]#562
Conversation
LawnGnome
left a comment
There was a problem hiding this comment.
One semi-related thought: right now we have a fairly half-arsed attempt at a non-fancy mode in output to handle the case where the output isn't going directly to a terminal. I'm thinking we might want to remove that and use this as the default once we have this fully implemented.
Ah, yes. I forgot about that. Question is though: is the text output for human or machine consumption? Because what's in here is definitely for machines. |
More human, but I'm not super convinced it's actually very useful for that. |
|
@sourcegraph/batchers As discussed yesterday: I want to get this in. I have some ideas on how to reduce the code duplication, but since that would require touching the old code I'd rather add a bit of duplication for now to get something running fast than to touch and possibly break the old code. For that I want to take my time. |
src batch [apply|preview]
…view]` (#562) * Start hacking on text-only output * Get stupidest text only output working * Print JSON objects per line when logging text * Remove last use of output in text-only mode * Update CHANGELOG
This adds a
-text-onlyflag tosrc batch [apply|preview]that is for internal use only.It's a highly-experimental feature that should not be used if you're not sure about what you're doing.