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

Bump minimum supported iOS version to 12.0 #27

Merged
merged 2 commits into from
Oct 7, 2020

Conversation

NickEntin
Copy link
Collaborator

@NickEntin NickEntin commented Oct 2, 2020

Resolves #25.

Copy link
Collaborator

@seanho seanho left a comment

Choose a reason for hiding this comment

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

Onward and upward

@NickEntin
Copy link
Collaborator Author

Realized there were a fair number of availability checks for iOS 11 that can be removed now. Addressed in 54c645d.

@NickEntin NickEntin force-pushed the entin/ios-12-minimum branch from 54c645d to b7faefa Compare October 7, 2020 00:50
@NickEntin
Copy link
Collaborator Author

No idea why that CI build had some differences in the iOS 13 snapshots the first time.

testTabBars()
- Item B: 3 items. Tab. 2 of 4.
+ Item B. Tab. 2 of 4.
- Item C: A. Tab. 3 of 4.
+ Item C. Tab. 3 of 4.

testStepper()
- Decrement. Button.
+ minus. Button.
- Increment. Button.
+ plus. Button.

testStepperAtMin()
- Decrement. Dimmed. Button.
+ minus. Dimmed. Button.
- Increment. Button.
+ plus. Button.

Kicking off a new build fixed it though. Same environment as far as I can tell. And everything seems to be working locally. 🤷‍♂️

I'm gonna call that a weird flake and move forward with this. I'll keep an eye on it and see if any more fail like this going forward though.

@NickEntin NickEntin merged commit eca961f into cashapp:master Oct 7, 2020
@NickEntin NickEntin deleted the entin/ios-12-minimum branch October 7, 2020 03:43
eliperkins added a commit to eliperkins/AccessibilitySnapshot that referenced this pull request Dec 19, 2022
These were leftover from cashapp#27 and are no longer used.
eliperkins added a commit to eliperkins/AccessibilitySnapshot that referenced this pull request Dec 19, 2022
These were leftover from cashapp#27 and are no longer used.
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.

Bump minimum supported iOS version to 12.0
3 participants