fresh-cat-90827
05/15/2022, 2:02 PMisort
doesn’t support saving the output error messages into a file (in the same way you can do flake8 --output-file=reports/flake8.txt
). If I’d be calling isort
on its own, I could just tee
, but since it happens in the ./pants lint
call, I can’t do that. Have I overlooked anything in the options and it’s actually possible to produce a file report? Is it something we could extend in Pants itself to produce such a report?hundreds-father-404
05/15/2022, 2:04 PMbitter-ability-32190
05/15/2022, 2:05 PMfresh-cat-90827
05/15/2022, 2:07 PM./pants lint ::
in CI, then I’d like to post errors as a comment. That’s what I do with the formatting and linting issues found. I’ve taken this approach, because reading through the CI output is not a great DevX imho 🙂 hence the need for a file I could read and then post errors as commentshundreds-father-404
05/15/2022, 2:09 PMfresh-cat-90827
05/15/2022, 2:11 PMfresh-cat-90827
05/15/2022, 2:21 PMbitter-ability-32190
05/15/2022, 2:37 PMbitter-ability-32190
05/15/2022, 2:37 PMbitter-ability-32190
05/15/2022, 2:38 PMloud-laptop-17949
05/16/2022, 5:43 PMloud-laptop-17949
05/16/2022, 5:43 PMFYI that is roughly what Toolchain's BuildSense does@hundreds-father-404 we use buildsense and I haven't seen this feature?
hundreds-father-404
05/16/2022, 5:52 PMwe use buildsense and I haven't seen this feature?In that you can look at the outputs in the web UI in a sanitized way, that it will filter out each tool etc. That is, you don't have to look at the long CI log if you don't want Doesn't add comments for you, tho (yet at least)
hundreds-father-404
05/16/2022, 5:52 PMIt would be great to add pants support for teeing that output to a file for processing.I think it makes sense. As with most Pants features, the hard part is figuring out the design, like what to name the option. Implementation should be pretty trivial from there
loud-laptop-17949
05/16/2022, 5:53 PMtee_stdout
and tee_stdin
make sense to me, but only if you know what tee
doeshundreds-father-404
05/16/2022, 5:54 PMredirect_stdout
? write_stdout
? save_stdout
?
(and do we have dedicated stdout vs stderr options, or we do both? or each tool decides?)loud-laptop-17949
05/16/2022, 5:54 PMloud-laptop-17949
05/16/2022, 5:55 PMloud-laptop-17949
05/16/2022, 5:55 PM