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

Split up table view thrash tests #1228

Closed
wants to merge 4 commits into from
Closed

Conversation

mikezucc
Copy link
Contributor

@mikezucc mikezucc commented Nov 9, 2018

Enabled and split the table view thrash test into a test file and a utility. This is part of an effort to write thrash tests for the collection view as well in order to try and reproduce crazy elusive texture/UIKit sync issues

@mikezucc
Copy link
Contributor Author

mikezucc commented Nov 9, 2018

Lol good catch by that bot 👍

@ghost
Copy link

ghost commented Nov 9, 2018

🚫 CI failed with log

@mikezucc
Copy link
Contributor Author

Fixed the source header 👍

@mikezucc
Copy link
Contributor Author

ping @maicki How does this look now?

@mikezucc
Copy link
Contributor Author

Pushed a commit to reenable those table view thrash tests

@ghost
Copy link

ghost commented Nov 19, 2018

1 Warning
⚠️ This is a big PR, please consider splitting it up to ease code review.

Generated by 🚫 Danger

@mikezucc
Copy link
Contributor Author

Closing this in favor of a new PR

@mikezucc mikezucc closed this Nov 20, 2018
@mikezucc
Copy link
Contributor Author

New PR #1246

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.

1 participant