Skip to content

don't add platform after tns platform clean is ran #2502

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

Closed
wants to merge 2 commits into from

Conversation

Plamen5kov
Copy link
Contributor

see more info here.
In short when tns platform clean <platform> is ran, the command will now only clean the platform, instead of removing it and then adding the latest platform package.

Copy link
Contributor

@rosen-vladimirov rosen-vladimirov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

According to

tns platform clean -h

The command does:

Removes and adds the selected platform to the project currently targets.

We have to either update the help content or just add the same version of the platform (as described in the related issue:

When I do a "clean" I expect to maintain the SAME version of the runtimes

@Plamen5kov Plamen5kov self-assigned this Feb 9, 2017
@Plamen5kov
Copy link
Contributor Author

Plamen5kov commented Feb 9, 2017

@rosen-vladimirov I'll update the documentation, but we don't need to add a platform. The platform will remain the same in the package.json so any command will install the correct platform. No need for clean to do it.

Edit: Need to change logic so cli respects what's in the package.json/nativescript/tns-android/ios version.

@rosen-vladimirov
Copy link
Contributor

I believe we no longer need these changes.

@Plamen5kov
Copy link
Contributor Author

Closing because of: #2615

@Plamen5kov Plamen5kov closed this Mar 17, 2017
@Plamen5kov Plamen5kov deleted the plamen5kov/dont_add_platform_on_clean branch March 17, 2017 07:03
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.

2 participants