Bump ios minimum deployment target - New PR#1292
Conversation
|
@justinseanmartin Hmm, 3 checks were canceled almost imminently after creating the PR. Logs are unavailable to me so I'm not able to see what's going on. |
|
Just needed approval to run. I hit the button on this, will see what comes out of the results. Thanks for picking this up! |
|
First attempt failed with: build (13.0, name=iPad Pro (12.9-inch) (5th generation),OS=15.0) build_xcode14 (14.1, name=iPad Pro (12.9-inch) (5th generation),OS=16.1) I'm re-running to see if these are just flakes, or if they're consistently broken in the same way. |
|
Must've been flakes, because everything passed on re-run. Let me take a look at the code changes. |
justinseanmartin
left a comment
There was a problem hiding this comment.
Thanks for the PR! The repo readme should probably say iOS 11+ instead of 11-13, but your change at least makes what we report in the podspec align with the repo readme, which is great.
New PR requested by @justinseanmartin
Original PR by @danielob #1283