[
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`
[][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: [email protected]
For additional commands, e-mail: [email protected]
