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

Fix missing copy #107

Merged
merged 4 commits into from
Aug 6, 2020
Merged

Fix missing copy #107

merged 4 commits into from
Aug 6, 2020

Conversation

HarelM
Copy link
Contributor

@HarelM HarelM commented Jul 23, 2020

It seems that when the folders plugins and platforms do not exists (as is usually in a CI scenario) the copy script does not run. This seems like a bug in cordova, but I believe this simple workaround is currently enough.
This should address #82.

@HarelM
Copy link
Contributor Author

HarelM commented Aug 3, 2020

@maintainers Anyone?

hooks/iosAddTarget.js Show resolved Hide resolved
@j3k0 j3k0 merged commit d6a2bf2 into j3k0:master Aug 6, 2020
@HarelM
Copy link
Contributor Author

HarelM commented Aug 6, 2020

Thanks!!
Let me know when a new version will be available in npm so I'll be able to remove the reference to my fork.

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.

2 participants