Hi, Thx. For e.g. pep8 job it looks like super useful as we can now see pep8 issues directly commented in PS, like e.g. at https://review.opendev.org/#/c/716773/1/neutron/tests/fullstack/test_securit... :)

On 1 Apr 2020, at 17:51, Monty Taylor <mordred@inaugust.com> wrote:

Hey everybody,

Yesterday mnaser finished up a long-standing TODO list item we had of leveraging Zuul’s ability to leave inline comments on changes by parsing out things like linter output and dropping them on changes. This is now live.

We’ve run in to a few gotchas (turns out there’s a lot of people doing a lot of different things) - all of which we’ve either fixed or have fixes in flight for. Notably there is a usage pattern of running pylint but only partially caring about the results, which turns inline comments from pylint output into complete noise. We’ve turned off the inline comments in openstack-tox-pylint:

https://review.opendev.org/#/c/716599/ https://review.opendev.org/#/c/716600/

although if your project uses it and would like inline comments from it, they can be re-enabled in your project. Similarly the same flag can be used to disable inline comments if your project decides they don't want them for some reason.

Work is under way to add parsing for Sphinx output and golangci-lint output.

If anybody runs in to any issues - like the results are too noisy or something is breaking where it shouldn’t, please let us know and we’ll get on it as quickly as possible.

Thanks! Monty

— Slawek Kaplonski Senior software engineer Red Hat