Skip to content

Init command should not force tns-core-modules version #1356

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Dec 17, 2015

Conversation

rosen-vladimirov
Copy link
Contributor

Currently tns init command forces the application to use the default version of tns-core-modules.
For example in case your CLI is version 1.4.3, tns init in non-interactive terminal will set version 1.4.0 (latest 1.4.x) of tns-core-modules even in case you already have defined your version in your package.json
Use --tnsCoreModules option by default - if it is specified, it has highest priority.
In case the console is not interactive, check if package.json has entry in dependencies section for tns-core-modules and use it.
In case the console is interactive, show all available versions to the user and let him pick up the one he wants.
In case the console is not interactive and the package.json does not have entry for tns-core-modules, use the default logic.

Targeting release branch as this feature is required for AppBuilder integration.

Currently `tns init` command forces the application to use the default version of tns-core-modules.
For example in case your CLI is version 1.4.3, `tns init` in non-interactive terminal will set version 1.4.0 (latest 1.4.x) of tns-core-modules even in case you already have defined your version in your package.json
Use --tnsCoreModules option by default - if it is specified, it has highest priority.
In case the console is not interactive, check if package.json has entry in dependencies section for tns-core-modules and use it.
In case the console is interactive, show all available versions to the user and let him pick up the one he wants.
In case the console is not interactive and the package.json does not have entry for `tns-core-modules`, use the default logic.
@rosen-vladimirov rosen-vladimirov self-assigned this Dec 16, 2015
@dtopuzov
Copy link
Contributor

👍

1 similar comment
@Mitko-Kerezov
Copy link
Contributor

👍

@ns-bot
Copy link

ns-bot commented Dec 17, 2015

Test PASSed.

rosen-vladimirov added a commit that referenced this pull request Dec 17, 2015
…mmand

Init command should not force tns-core-modules version
@rosen-vladimirov rosen-vladimirov merged commit c30d685 into release Dec 17, 2015
@rosen-vladimirov rosen-vladimirov deleted the vladimirov/improve-init-command branch December 17, 2015 07:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants