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

CB-13523: Add flag for Xcode-managed provisioning #347

Merged
merged 1 commit into from Nov 15, 2017

Conversation

dpogue
Copy link
Member

@dpogue dpogue commented Oct 31, 2017

Platforms affected

iOS with Xcode 9

What does this PR do?

This adds a automaticProvisioning option to the build config which passes flags to xcodebuild during the exporting step that allow it to automatically create and update provisioning profiles for automatic code signing.

I don't love that we're adding even more random stuff to the build config, but I suspect most people don't want this to be enabled by default.

What testing has been done on this change?

Added a case to existing spec tests.

Tested with a Cordova project at work and confirmed to be working.

Checklist

  • Reported CB-13523
  • Commit message follows the format
  • Added automated test coverage as appropriate for this change.

@codecov-io
Copy link

codecov-io commented Nov 1, 2017

Codecov Report

Merging #347 into master will decrease coverage by 0.01%.
The diff coverage is 40%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #347      +/-   ##
==========================================
- Coverage   63.43%   63.41%   -0.02%     
==========================================
  Files          14       14              
  Lines        1690     1692       +2     
  Branches      283      284       +1     
==========================================
+ Hits         1072     1073       +1     
- Misses        618      619       +1
Impacted Files Coverage Δ
bin/templates/scripts/cordova/lib/build.js 36.44% <40%> (+0.12%) ⬆️

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 aaf229b...110b084. Read the comment docs.

@dpogue dpogue force-pushed the automatic-provisioning branch 3 times, most recently from 88f9fe7 to 9bd5306 Compare November 3, 2017 03:32
This adds a `automaticProvisioning` option to the build config which
passes flags to xcodebuild during the exporting step that allow it to
automatically create and update provisioning profiles for automatic code
signing.
@dpogue dpogue changed the title CB-13523: Add flag for Xcode-managed provisioning (DON'T MERGE) CB-13523: Add flag for Xcode-managed provisioning Nov 4, 2017
@janpio
Copy link
Member

janpio commented Nov 10, 2017

Nice.

(Side note: Please make sure that the new option also gets documented - these definitely need documentation)

@surajpindoria surajpindoria merged commit 5cb8bd1 into apache:master Nov 15, 2017
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.

None yet

4 participants