Post lint results to Lando
Categories
(Developer Infrastructure :: Source Code Analysis, enhancement)
Tracking
(Not tracked)
People
(Reporter: marco, Assigned: andi)
References
Details
Attachments
(1 file)
In bug 1699137, we're introducing a new API in Lando to upload warnings the user must ack before landing.
We should use this API in the code review bot to upload lint results to Lando in addition to Phabricator.
Comment 1•2 years ago
|
||
We should also post non-lint failures as well as a warning when tasks are unfinished (maybe the latter should be a separate check box?)
Reporter | ||
Comment 2•2 years ago
|
||
(In reply to Andrew Halberstadt [:ahal] from comment #1)
We should also post non-lint failures as well as a warning when tasks are unfinished (maybe the latter should be a separate check box?)
Yes, any failure will be posted (I called them "lint results" because that's the Phabricator nomenclature, but actually they can be anything).
Regarding unfinished tasks, with the API defined in bug 1699137 we'll send a first "build in progress" warning, then when the actual results come in we remove it and upload the actual failures (or nothing if there was no failure).
Reporter | ||
Updated•1 year ago
|
Reporter | ||
Comment 3•1 year ago
|
||
Assignee | ||
Comment 4•1 year ago
|
||
This has been done but we are waiting for the latest version of lando to be deployed.
Updated•6 months ago
|
Description
•