New issue
Advanced search Search tips

Issue 726878 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Jun 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 1
Type: Bug

Blocking:
issue 707006



Sign in to add a comment

[WPT Export] Gerrit CL -> PR logic isn't correctly filtering out expected.txt files

Project Member Reported by jeffcarp@chromium.org, May 26 2017

Issue description

This prevented https://chromium-review.googlesource.com/c/513527/ from having a provisional PR made, and possibly more CLs we aren't aware of.
 
Cc: jeffcarp@chromium.org
 Issue 730779  has been merged into this issue.
The CL that fixed this was crrev.com/c/540146/, which has landed but pointed to  issue 730779  which was merged into this one.

A CL that's a good example of when an -expected.txt file got included in the upstream PR is crrev.com/c/532595/. I just updated the commit message which will cause the exporter to upload a new patch which will hopefully not include the -expected.txt file. After I verify that, I'll close this bug.
Status: Fixed (was: Assigned)
Confirmed as fixed: https://github.com/w3c/web-platform-tests/pull/6222/files
Components: Blink>Infra>Ecosystem
Components: -Blink>Infra>Predictability

Sign in to add a comment