Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Guard photo library with macro for tests #1147

Merged

Conversation

wsdwsd0829
Copy link
Contributor

No description provided.

It will fail to build if photo library is disabled cause the test is
depending on it.
Copy link
Contributor

@maicki maicki left a comment

Choose a reason for hiding this comment

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

Cool

@ghost
Copy link

ghost commented Sep 28, 2018

🚫 CI failed with log

@nguyenhuy
Copy link
Member

CI was flaky. The commit with actual code changes passed, so I'm gonna land this.

@nguyenhuy
Copy link
Member

@wsdwsd0829 Thanks for the fix, btw.

@nguyenhuy nguyenhuy merged commit f656dbb into TextureGroup:master Sep 28, 2018
@wsdwsd0829
Copy link
Contributor Author

wsdwsd0829 commented Sep 28, 2018 via email

mikezucc pushed a commit to mikezucc/Texture that referenced this pull request Oct 2, 2018
* fix SIMULATE_WEB_RESPONSE not imported TextureGroup#449

* Fix to make rangeMode update in right time

* remove uncessary assert

* Fix collection cell editing bug for iOS 9 & 10

* Revert "Fix collection cell editing bug for iOS 9 & 10"

This reverts commit 06e18a1.

* Only test when photo library is enabled.

It will fail to build if photo library is disabled cause the test is
depending on it.

* Add ChangeLog.
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.

None yet

3 participants