Issue metadata
Sign in to add a comment
|
[WPT Export] Comment important PR status updates on downstream CLs |
||||||||||||||||||||||||
Issue descriptionVisibility into the export process is difficult since the exporter is running as a builder. We should comment status updates on the CL, such as: - Travis CI failed (this should CC blink-infra@) - The PR was closed - The PR was merged
,
Mar 30 2017
,
Jul 3 2017
,
Jul 3 2017
,
Aug 3 2017
This would make life easier for Chromium contributors, however it would be difficult to implement on the Exporter side because we'd need to make sure we aren't creating duplicate comments on a CL every 10 minutes. Historically since provisional PRs were introduced in issue 700092 , contributors have been very good about checking the upstream PR for updates.
,
Aug 6
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue. Sorry for the inconvenience if the bug really should have been left as Available. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Aug 6
FWIW, the "PR was merged" case has already been implemented, so that leaves us the first two cases, which are still good to have.
,
Aug 16
Merging this into issue 711447, as the only remaining item here is the notification of failed GitHub checks. |
|||||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||||
Comment 1 by jeffcarp@chromium.org
, Mar 13 2017