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

breaking: bump deployment-target to 11.0 #779

Merged
merged 1 commit into from Feb 10, 2020

Conversation

erisu
Copy link
Member

@erisu erisu commented Feb 10, 2020

Motivation and Context

The purpose of this PR is to bump the minimum deployment target to iOS 11+.

This PR prepares the repo with the requirements to resolve issues/problems that were introduced when dropping UIWebView for WKWebView.

One example is the strict CORS request guidelines the WKWebView enforces. CORS requests can not be made from the file://. To resolve this, the minimum deployment target must be 11+ before the solution can be implemented in a later PR.

Description

Bump the IPHONEOS_DEPLOYMENT_TARGET to 11.0

Testing

  • npm t
  • cordova platform add
  • cordova build ios

Checklist

  • I've run the tests to see all new and existing tests pass
  • I've updated the documentation if necessary

@erisu erisu added this to the 6.0.0 milestone Feb 10, 2020
@erisu erisu changed the title chore: bump deployment-target to 11.0 breaking: bump deployment-target to 11.0 Feb 10, 2020
@erisu erisu force-pushed the breaking/bump-deployment-target branch from 3d5c978 to 3c3d864 Compare February 10, 2020 05:08
@codecov-io
Copy link

codecov-io commented Feb 10, 2020

Codecov Report

Merging #779 into master will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #779   +/-   ##
=======================================
  Coverage   74.26%   74.26%           
=======================================
  Files          11       11           
  Lines        1815     1815           
=======================================
  Hits         1348     1348           
  Misses        467      467
Impacted Files Coverage Δ
bin/templates/scripts/cordova/lib/Podfile.js 73.58% <100%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 240a177...3c3d864. Read the comment docs.

@erisu erisu added this to In progress in Release Plan via automation Feb 10, 2020
@erisu erisu moved this from In progress to Review in progress in Release Plan Feb 10, 2020
Copy link
Member

@jcesarmobile jcesarmobile left a comment

Choose a reason for hiding this comment

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

Looks good

Release Plan automation moved this from Review in progress to Reviewer approved Feb 10, 2020
@NiklasMerz NiklasMerz self-requested a review February 10, 2020 08:59
Copy link
Member

@NiklasMerz NiklasMerz left a comment

Choose a reason for hiding this comment

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

LGTM, local tests pass

@erisu erisu merged commit 59d0dd7 into apache:master Feb 10, 2020
Release Plan automation moved this from Reviewer approved to Done Feb 10, 2020
@erisu erisu deleted the breaking/bump-deployment-target branch February 10, 2020 11:25
NiklasMerz pushed a commit to GEDYSIntraWare/cordova-ios that referenced this pull request Feb 24, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
Release Plan
  
Done
Development

Successfully merging this pull request may close these issues.

None yet

4 participants