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

Use promise-based result for Cordova #66

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

l3ender
Copy link

@l3ender l3ender commented Jun 16, 2019

Resolves #65.

See Cordova Javascript doc for sample/example.

This will allow the swift plugin to correctly run concurrently with other plugins that run on the same hook that also modify the pbxproj file.

@l3ender
Copy link
Author

l3ender commented Jun 17, 2019

Note, this isn't a very big change if you compare and ignore whitespace: https://github.com/akofman/cordova-plugin-add-swift-support/pull/66/files?w=1.

@l3ender l3ender mentioned this pull request Jun 17, 2019
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.

Plugin not correctly synchronized (race condition conflicts with other plugins)
1 participant