hundreds-father-404
07/31/2020, 8:55 PMProcess
has been completed or not? The new messages for “Searching for ‘python3’ on PATH` are really chattywitty-crayon-22786
07/31/2020, 9:03 PMhundreds-father-404
07/31/2020, 9:07 PMthey should be cached thoughYes, but I also think this is a dangerous mindset that we need to remind ourselves to watch out for. The first time experience of a user matters. For a first time user, are these particular messages pulling their weight? I think no.
we don’t have a way to set the “level” of a process… might be a quick fix.Would this be adding a property to
Process
called log level
? Then piping it through to the engine?witty-crayon-22786
07/31/2020, 9:13 PMwill open a new issue for that.https://github.com/pantsbuild/pants/issues/10526
hundreds-father-404
07/31/2020, 11:58 PMhappy-kitchen-89482
08/01/2020, 12:01 AMhundreds-father-404
08/01/2020, 12:01 AMprotoc
- extract an archive (archive.py
)
- run cloc
- search for binaries, like python
happy-kitchen-89482
08/01/2020, 12:01 AMhundreds-father-404
08/01/2020, 12:01 AMpytest
witty-crayon-22786
08/01/2020, 12:04 AMdescription
hundreds-father-404
08/01/2020, 12:05 AMwitty-crayon-22786
08/01/2020, 12:06 AMAgreed, but even if the level is set to Debug, it will still show up in the dynamic UI, right?yea, true.
hundreds-father-404
08/01/2020, 12:07 AMcloc
is redundant because we immediately after output the result of cloc
witty-crayon-22786
08/01/2020, 12:08 AMhundreds-father-404
08/01/2020, 12:09 AMwitty-crayon-22786
08/01/2020, 12:11 AM… i had almost forgotten about that.not even almost: “fully forgotten”.
TestResult
to dump the stderr in def message
if it’s a failurehundreds-father-404
08/01/2020, 12:16 AMwitty-crayon-22786
08/01/2020, 12:16 AM