1616218 - [Automated review] Reviewbot mozlint-eslint is missing information

Status: RESOLVED FIXED

(Developer Infrastructure :: Source Code Analysis, task)

Description

[Simon Giesecke [:sg] [he/him]]

Phabricator URL: https://phabricator.services.mozilla.com/D60706

The report says:

Code analysis found 3 defects in the diff 229978:

    3 defects found by mozlint-eslint

You can run this analysis locally with:

    ./mach lint --warnings path/to/file (JS/Python/etc)

This is not very helpful. It doesn't include the actual warnings, and it doesn't even mention the files the warnings refer to.

Comment 1

[Sylvestre Ledru [:Sylvestre]]

Yeah, it is confusing.
I guess it is because https://treeherder.mozilla.org/#/jobs?repo=try&revision=cc14533c21d109a94fa88a0f513fb54b829f5fee is still running.
Bastien, wdyt?

Comment 2

[Bastien Abadie [:bastien]]

Yes, Sylvestre is right: a new build has been triggered as you pushed a new diff.
Unfortunately Phabricator immediately removes the previous build failures, and you can't view them again in Phabricator.

But we have them stored in our backend, so you can view the 3 errors here: https://code-review.moz.tools/#/diff/229978

I'll add a link towards that page in the summary comment to avoid that issue in the future.

Comment 3

[Andi [:andi]]

This should be fixed now.