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

[DEPENDENCY] Update cordova-plugin-add-swift-support to v2.0.1 #155

Merged

Conversation

byronaltice
Copy link
Contributor

Description

Allows building in cordova 9. See: akofman/cordova-plugin-add-swift-support#56

How did you test your changes?

Force installed [email protected], and built successfully.

@byronaltice byronaltice requested a review from NiklasMerz as a code owner April 1, 2019 21:21
@byronaltice
Copy link
Contributor Author

Related issue: #153

Copy link
Owner

@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.

Please also update plugin.xml

@byronaltice
Copy link
Contributor Author

Please also update config.xml

I may be misunderstanding, config.xml does not exist in the project. Could you clarify please?

@NiklasMerz
Copy link
Owner

I am sorry. I mean plugin.xml. The dependency is set in package.jsonandplugin.xml`. Sorry for the confusion.

@byronaltice byronaltice force-pushed the feature/support-cordova-9 branch from 9c521ee to b29b02d Compare April 2, 2019 14:28
@byronaltice
Copy link
Contributor Author

No problem, updated. Please review. Thanks

@byronaltice byronaltice force-pushed the feature/support-cordova-9 branch from b29b02d to 39668ec Compare April 2, 2019 14:29
plugin.xml Show resolved Hide resolved
@l3ender
Copy link

l3ender commented Apr 3, 2019

Thanks for this fix. Are we able to merge and create a new release?

@NiklasMerz NiklasMerz merged commit 6cb20cb into NiklasMerz:master Apr 3, 2019
@NiklasMerz
Copy link
Owner

I merged it and you can use the master branch. I will try to get a new release out in the next few days.

@byronaltice byronaltice deleted the feature/support-cordova-9 branch April 3, 2019 15:28
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.

3 participants