[ https://issues.apache.org/jira/browse/CB-11117?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15258462#comment-15258462 ]
ASF GitHub Bot commented on CB-11117: ------------------------------------- Github user codecov-io commented on the pull request: https://github.com/apache/cordova-lib/pull/429#issuecomment-214812288 ## [Current coverage][cc-pull] is **100%** > Merging [#429][cc-pull] into [master][cc-base-branch] will increase coverage by **+19.90%** ```diff @@ master #429 diff @@ ======================================== Files 69 69 Lines 5027 9878 +4851 Methods 0 841 +841 Branches 971 1271 +300 ======================================== + Hits 4027 9878 +5851 + Misses 995 0 -995 + Partials 5 0 -5 ``` 1. 5 files in `...dova-lib/src/cordova` were modified. [more](https://codecov.io/gh/apache/cordova-lib/commit/3abe8b389628cc0256f16fa6d66da7a460485925/changes?src=pr#636F72646F76612D6C69622F7372632F636F72646F7661) - Misses `-16` - Hits `+167` 1. 2 files (not in diff) in `...lib/src/util/windows` were modified. [more](https://codecov.io/gh/apache/cordova-lib/commit/3abe8b389628cc0256f16fa6d66da7a460485925/changes?src=pr#636F72646F76612D6C69622F7372632F7574696C2F77696E646F7773) - Misses `-52` - Partials `-1` - Hits `+383` 1. 3 files (not in diff) in `cordova-lib/src/util` were modified. [more](https://codecov.io/gh/apache/cordova-lib/commit/3abe8b389628cc0256f16fa6d66da7a460485925/changes?src=pr#636F72646F76612D6C69622F7372632F7574696C) - Misses `-10` - Hits `+55` 1. 7 files (not in diff) in `...lib/src/plugman/util` were modified. [more](https://codecov.io/gh/apache/cordova-lib/commit/3abe8b389628cc0256f16fa6d66da7a460485925/changes?src=pr#636F72646F76612D6C69622F7372632F706C75676D616E2F7574696C) - Misses `-19` - Hits `+197` 1. 3 files (not in diff) in `...src/plugman/registry` were modified. [more](https://codecov.io/gh/apache/cordova-lib/commit/3abe8b389628cc0256f16fa6d66da7a460485925/changes?src=pr#636F72646F76612D6C69622F7372632F706C75676D616E2F7265676973747279) - Misses `-9` - Hits `+142` 1. 7 files (not in diff) in `...rc/plugman/platforms` were modified. [more](https://codecov.io/gh/apache/cordova-lib/commit/3abe8b389628cc0256f16fa6d66da7a460485925/changes?src=pr#636F72646F76612D6C69622F7372632F706C75676D616E2F706C6174666F726D73) - Misses `-84` - Hits `+608` 1. 12 files (not in diff) in `...dova-lib/src/plugman` were modified. [more](https://codecov.io/gh/apache/cordova-lib/commit/3abe8b389628cc0256f16fa6d66da7a460485925/changes?src=pr#636F72646F76612D6C69622F7372632F706C75676D616E) - Misses `-261` - Hits `+1202` 1. 2 files (not in diff) in `...va-lib/src/platforms` were modified. [more](https://codecov.io/gh/apache/cordova-lib/commit/3abe8b389628cc0256f16fa6d66da7a460485925/changes?src=pr#636F72646F76612D6C69622F7372632F706C6174666F726D73) - Misses `-12` - Partials `-2` - Hits `+243` 1. 3 files (not in diff) in `...ordova-lib/src/hooks` were modified. [more](https://codecov.io/gh/apache/cordova-lib/commit/3abe8b389628cc0256f16fa6d66da7a460485925/changes?src=pr#636F72646F76612D6C69622F7372632F686F6F6B73) - Misses `-48` - Hits `+208` 1. 2 files (not in diff) in `...etadata/parserhelper` were modified. [more](https://codecov.io/gh/apache/cordova-lib/commit/3abe8b389628cc0256f16fa6d66da7a460485925/changes?src=pr#636F72646F76612D6C69622F7372632F636F72646F76612F6D657461646174612F70617273657268656C706572) - Hits `+64` [![Sunburst](https://codecov.io/gh/apache/cordova-lib/pull/429/graphs/sunburst.svg?size=660&src=pr)][cc-pull] > Powered by [Codecov](https://codecov.io?src=pr). Last updated by 3abe8b3 [cc-base-branch]: https://codecov.io/gh/apache/cordova-lib/branch/master?src=pr [cc-pull]: https://codecov.io/gh/apache/cordova-lib/pull/429?src=pr > Preparing platforms should skip copying files which haven't changed > ------------------------------------------------------------------- > > Key: CB-11117 > URL: https://issues.apache.org/jira/browse/CB-11117 > Project: Apache Cordova > Issue Type: Improvement > Components: Android, iOS, Windows > Reporter: Jason Ginchereau > Assignee: Jason Ginchereau > > Many cordova CLI commands include a "prepare" operation, including 'cordova > build', 'cordova run', 'cordova plugin add', and more. Every time each of > those commands runs, the target platform is "prepared", which involves > copying all files from the [<project>/www, > <project>/platforms/<platform>/platform_www, <project>/merges/<platform>] to > the platform's target www folder, as well as copying a bunch of icons and > splash screens to platform-specific locations. > For the very first prepare of a platform, all that file copying is necessary. > But most of the time after that most of the files being copied have not > changed and therefore don't really need to be copied again. So the typical > developer inner loop (edit a few source files, build and run the app, repeat) > is a lot slower than it could be for a Cordova project, especially one that > includes a significant number of source files or resources. > Instead, Cordova should be smart enough to skip copying of files that haven't > changed, based on their last-modified timestamp. (But also there should still > be a way to force a clean/full/non-incremental build if desired.) -- This message was sent by Atlassian JIRA (v6.3.4#6332) --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@cordova.apache.org For additional commands, e-mail: issues-h...@cordova.apache.org