@dpogue 

**The CocoaPods improvement** is done and can be merged in now. 
`cordova-common` needs to be merged first. Then we need to release 
cordova-common next major. Once its tagged, then we need to update 
`cordova-ios` package.json and then merge in the chanfges.

Is the **Android Gradle's latest version** actually a requirement for this next 
major? Without the latest version, does Android P builds fails? This task seems 
new and unexpectedly added.

I already reviewed **Updating Android gradle files to not use deprecated 
commands** and said that the changes were OK. The question I left as a comment 
only asked if we should allow developers the flexibility to pick between 
`implementation` and `api`. Adding the support is not required for next major. 

I knew there were **other Gradle PRs** that had questions about adding the 
helper methods into the dependencies scope to better isolate features that not 
everyone uses for example Kotlin and EnableGoogleServicesPlugin.
https://github.com/apache/cordova-android/pull/438
https://github.com/apache/cordova-android/pull/441

[ Full content available at: https://github.com/apache/cordova/issues/10 ]
This message was relayed via gitbox.apache.org for [email protected]

Reply via email to