[ https://issues.apache.org/jira/browse/CB-11244?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16049513#comment-16049513 ]
ASF GitHub Bot commented on CB-11244: ------------------------------------- Github user codecov-io commented on the issue: https://github.com/apache/cordova-android/pull/384 # [Codecov](https://codecov.io/gh/apache/cordova-android/pull/384?src=pr&el=h1) Report > Merging [#384](https://codecov.io/gh/apache/cordova-android/pull/384?src=pr&el=desc) into [master](https://codecov.io/gh/apache/cordova-android/commit/3a6e898b12eac31dfc16a4c526dfba8fab158723?src=pr&el=desc) will **decrease** coverage by `0.29%`. > The diff coverage is `20.58%`. [![Impacted file tree graph](https://codecov.io/gh/apache/cordova-android/pull/384/graphs/tree.svg?token=q14nMf6C5a&src=pr&width=650&height=150)](https://codecov.io/gh/apache/cordova-android/pull/384?src=pr&el=tree) ```diff @@ Coverage Diff @@ ## master #384 +/- ## ========================================= - Coverage 39.52% 39.22% -0.3% ========================================= Files 16 16 Lines 1551 1555 +4 Branches 277 279 +2 ========================================= - Hits 613 610 -3 - Misses 938 945 +7 ``` | [Impacted Files](https://codecov.io/gh/apache/cordova-android/pull/384?src=pr&el=tree) | Coverage Δ | | |---|---|---| | [bin/templates/cordova/lib/builders/builders.js](https://codecov.io/gh/apache/cordova-android/pull/384?src=pr&el=tree#diff-YmluL3RlbXBsYXRlcy9jb3Jkb3ZhL2xpYi9idWlsZGVycy9idWlsZGVycy5qcw==) | `33.33% <ø> (ø)` | :arrow_up: | | [...n/templates/cordova/lib/builders/GenericBuilder.js](https://codecov.io/gh/apache/cordova-android/pull/384?src=pr&el=tree#diff-YmluL3RlbXBsYXRlcy9jb3Jkb3ZhL2xpYi9idWlsZGVycy9HZW5lcmljQnVpbGRlci5qcw==) | `28.84% <0%> (-1.3%)` | :arrow_down: | | [bin/templates/cordova/lib/build.js](https://codecov.io/gh/apache/cordova-android/pull/384?src=pr&el=tree#diff-YmluL3RlbXBsYXRlcy9jb3Jkb3ZhL2xpYi9idWlsZC5qcw==) | `10.13% <0%> (-0.21%)` | :arrow_down: | | [bin/templates/cordova/lib/device.js](https://codecov.io/gh/apache/cordova-android/pull/384?src=pr&el=tree#diff-YmluL3RlbXBsYXRlcy9jb3Jkb3ZhL2xpYi9kZXZpY2UuanM=) | `18.18% <0%> (ø)` | :arrow_up: | | [bin/templates/cordova/lib/AndroidStudio.js](https://codecov.io/gh/apache/cordova-android/pull/384?src=pr&el=tree#diff-YmluL3RlbXBsYXRlcy9jb3Jkb3ZhL2xpYi9BbmRyb2lkU3R1ZGlvLmpz) | `94.73% <100%> (ø)` | :arrow_up: | | [...in/templates/cordova/lib/builders/GradleBuilder.js](https://codecov.io/gh/apache/cordova-android/pull/384?src=pr&el=tree#diff-YmluL3RlbXBsYXRlcy9jb3Jkb3ZhL2xpYi9idWlsZGVycy9HcmFkbGVCdWlsZGVyLmpz) | `19.62% <15.78%> (-0.38%)` | :arrow_down: | | [bin/templates/cordova/Api.js](https://codecov.io/gh/apache/cordova-android/pull/384?src=pr&el=tree#diff-YmluL3RlbXBsYXRlcy9jb3Jkb3ZhL0FwaS5qcw==) | `42.37% <33.33%> (-0.61%)` | :arrow_down: | ------ [Continue to review full report at Codecov](https://codecov.io/gh/apache/cordova-android/pull/384?src=pr&el=continue). > **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta) > `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data` > Powered by [Codecov](https://codecov.io/gh/apache/cordova-android/pull/384?src=pr&el=footer). Last update [3a6e898...49b76f5](https://codecov.io/gh/apache/cordova-android/pull/384?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments). > Update Android Project Structure to be more compatible with Android Studio > -------------------------------------------------------------------------- > > Key: CB-11244 > URL: https://issues.apache.org/jira/browse/CB-11244 > Project: Apache Cordova > Issue Type: Improvement > Components: cordova-android > Reporter: Joe Bowser > Assignee: Joe Bowser > Labels: CordovaAndroidSix > > Currently our existing Android projects have the old Android Project > structure created by default, even for new projects, which makes no sense > since we no longer support Eclipse as the default project editor and have > moved to Android Studio for Android development. > The main reason for doing this is to clean up dependencies and to allow for > projects and dependencies to work more like how modern Android projects work > and to remove as much custom Gradle code as possible. This would also allow > us in the future to move towards having Android Plugin code work as Android > Libraries with Resources instead of just copying things across, which gives > us the ability to add JUnit tests and bundled resources. This would > dramatically increase the quality of plugins such as InAppBrowser, and third > party plugins such as the Barcode Scanner. > This would have to be done on the next major version, and the upgrade would > be tricky to do. However, the benefits at this point would greatly outweigh > the costs of maintaining the old project structure. The old cordova-common > code in Cordova-Android 5.0.x would allow for plugins in the short term to > work with both projects until we get the new project structure ready. -- This message was sent by Atlassian JIRA (v6.4.14#64029) --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@cordova.apache.org For additional commands, e-mail: issues-h...@cordova.apache.org