-
-
Notifications
You must be signed in to change notification settings - Fork 409
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
refactor: Remove idevicescreenshot since don't need it #1022
Conversation
8b6a518
to
9b3fe5a
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is WDA's screenshotting now stable enough?
Let me do a load test |
@imurchie WDA screenshot code doesn't work with iOS versions below 11. I need to fix that first apparently but then it is reliable |
Alrighty. Should we hold off on this until that can be done? |
@umutuzgur Appium does not support Xcode8 (and iOS10, which is based on it) since quite a while. |
@imurchie Can you confirm this? I should probably communicate this internally then |
9b3fe5a
to
9fdefa8
Compare
@imurchie The fix is merged in WDA. All looks good now and we can proceed with this PR |
I'm suggesting removing
idevicescreenshot
if it is no longer needed