Skip to content

Conversation

@jason-simmons
Copy link
Member

@jason-simmons jason-simmons commented May 1, 2020

New Windows engine binaries built with Clang result in diffs of a few
pixels when running this test. The test will be disabled until the
engine roll with these changes lands and the test can be updated to
point to the new golden images.

@jason-simmons jason-simmons requested a review from Hixie May 1, 2020 18:05
@Hixie
Copy link
Contributor

Hixie commented May 1, 2020

If this is just temporary (for a few hours) we should just rename it to ....test.disabled and add a comment pointing to why we are disabling it, rather than deleting it.

LGTM.

Hixie
Hixie previously approved these changes May 1, 2020
New Windows engine binaries built with Clang result in diffs of a few
pixels when running this test.  The test will be disabled until the
engine roll with these changes lands and the test can be updated to
point to the new golden images.
@jason-simmons
Copy link
Member Author

Updated the test file name

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants