Skip to content

Conversation

tomzig16
Copy link
Contributor

Purpose of this PR

Filters were added to the master branch in commit d26e9d1, together with Android support. However, as now I am more focused in making iOS run and have been blocked a bit on Android backport I decided to push PR for filters alone for now.

Testing status

Manual Tests

Locally tested if filters out (tested on OSX platform)

Automated Tests

Tested yamato with several configurations (DX11 playmode)

Links

Yamato: https://yamato.prd.cds.internal.unity3d.com/jobs/902-Graphics/tree/7.x%252Fbackport%252Ffilters

Comments to reviewers

This PR does not change any tests themselves.

@tomzig16 tomzig16 requested a review from a team as a code owner May 13, 2020 16:57
Copy link
Contributor

@sophiaaar sophiaaar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm fine to approve this as it is needed, but this may end up getting overwritten when my backport for the graphics test framework update lands

@sophiaaar
Copy link
Contributor

my backport PR is nearly ready btw #474

@sophiaaar
Copy link
Contributor

test filters landed as part of #531, closing this PR

@sophiaaar sophiaaar closed this May 19, 2020
@sophiaaar sophiaaar deleted the 7.x/backport/filters branch May 19, 2020 11:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants