[ 
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

Reply via email to