cordova-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF GitHub Bot (JIRA)" <>
Subject [jira] [Commented] (CB-11412) Update template implementation in cordova create --template
Date Tue, 14 Jun 2016 22:42:30 GMT


ASF GitHub Bot commented on CB-11412:

Github user stevengill commented on a diff in the pull request:
    --- Diff: cordova-lib/spec-cordova/create.spec.js ---
    @@ -121,6 +122,36 @@ describe('create end-to-end', function() {
             // expect('TestBase');
    +    function checkSubDir() {
    +        // Check if top level dirs exist.
    +        var dirs = ['hooks', 'platforms', 'plugins', 'www'];
    +        dirs.forEach(function(d) {
    +            expect(path.join(project, d)).toExist();
    +        });
    +        expect(path.join(project, 'hooks', '')).toExist();
    +        //index.js and template folder should not exist (inner files should be copied
to the project folder)
    +        expect(path.join(project, 'index.js')).not.toExist();
    +        expect(path.join(project, 'template')).not.toExist();
    --- End diff --
    Is this assuming the template folder is named `template`?

> Update template implementation in cordova create --template
> -----------------------------------------------------------
>                 Key: CB-11412
>                 URL:
>             Project: Apache Cordova
>          Issue Type: Improvement
>          Components: CordovaLib
>            Reporter: Caryn Tran
>            Assignee: Caryn Tran
> Sometimes we do want a `package.json` copied over. `package.json` could have build scripts
that are essential for the template to work. To handle this use case, template authors are
suggested to put template files in a subdirectory and point to that subdirectory in index.js.
All files in the subdirectory are copied over.
> Update the current template implementation to ignore the list of files above that shouldn't
be copied over when using `--template`. If a subdirectory is used, copy the entire contents
of the subdirectory.
> If a `package.json` exists in the subdirectory, we should update its fields like we do
to `config.xml`.
Only field to update in `package.json` should be ``. `` should be
updated to use `` lowercased (npm package names have to be lowercase). ``
is the optional third argument `cordova create` takes.

This message was sent by Atlassian JIRA

To unsubscribe, e-mail:
For additional commands, e-mail:

View raw message