Mailing-List: contact dev-help@cordova.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@cordova.apache.org
From: stevengill <git@git.apache.org>
To: dev@cordova.apache.org
Reply-To: dev@cordova.apache.org
References: <git-pr-287-cordova-cli@git.apache.org>
In-Reply-To: <git-pr-287-cordova-cli@git.apache.org>
Subject: [GitHub] cordova-cli pull request #287: CB-13303 : added noprod/production as an opti...
Content-Type: text/plain
Message-Id: <20171003213356.2657DF56FF@git1-us-west.apache.org>
Date: Tue,  3 Oct 2017 21:33:56 +0000 (UTC)
archived-at: Tue, 03 Oct 2017 21:33:58 -0000

Github user stevengill commented on a diff in the pull request:

    https://github.com/apache/cordova-cli/pull/287#discussion_r142529980
  
    --- Diff: src/cli.js ---
    @@ -465,6 +473,14 @@ function cli (inputArgs) {
                 // User explicitly did not pass in searchpath
                 args.searchpath = conf.get('searchpath');
             }
    +        if (args.production === undefined) {
    +            // User explicitly did not pass in noprod
    +            args.production = conf.get('production');
    +        }
    +        if (args.noprod === undefined) {
    --- End diff --
    
    lets not offer `noprod` in config. Just `args.production`. We only need noprod like you have in line 454. Used to figure out of args.production is true or false. 


---

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@cordova.apache.org
For additional commands, e-mail: dev-help@cordova.apache.org