[
https://issues.apache.org/jira/browse/CB-11262?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15286300#comment-15286300
]
ASF GitHub Bot commented on CB-11262:
-------------------------------------
Github user daserge commented on a diff in the pull request:
https://github.com/apache/cordova-cli/pull/251#discussion_r63492417
--- Diff: src/cli.js ---
@@ -234,17 +234,24 @@ function cli(inputArgs) {
var args = nopt(knownOpts, shortHands, inputArgs);
- if (args.version) {
- var cliVersion = require('../package').version;
+ var cliVersion = require('../package').version;
+ // TODO: Use semver.prerelease when it gets released
+ var usingPrerelease = /-nightly|-dev$/.exec(cliVersion);
+ if (args.version || usingPrerelease) {
var libVersion = require('cordova-lib/package').version;
var toPrint = cliVersion;
if (cliVersion != libVersion || /-dev$/.exec(libVersion)) {
toPrint += ' (cordova-lib@' + libVersion + ')';
}
- console.log(toPrint);
- return Q();
- }
+ if (args.version) {
+ console.log(toPrint);
+ return Q();
+ } else {
+ // Show a warning and continue
+ console.log('Warning: using prerelease version ' + toPrint);
--- End diff --
Logger is being initialized below that's why I'm not using it.
I agree regarding the `warn` visibility though - will try to refactor this,
thanks!
> Add a warning about prerelease lib/cli usage
> --------------------------------------------
>
> Key: CB-11262
> URL: https://issues.apache.org/jira/browse/CB-11262
> Project: Apache Cordova
> Issue Type: Task
> Components: CLI
> Reporter: Sergey Shakhnazarov
> Assignee: Sergey Shakhnazarov
>
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
