Skip to content

NS-CLI Readme #78

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 6 commits into from
Aug 29, 2014
Merged

NS-CLI Readme #78

merged 6 commits into from
Aug 29, 2014

Conversation

ikoevska
Copy link
Contributor

  • Readme.md
  • Contributing.md (Note that Contributing.md is borrowed from the AB CLI, so you might want to change it. This is just a placeholder for now).

Iva Koevska added 3 commits August 27, 2014 20:37
Adding Run section to indicate that the run command exists
@justcodebuilduser
Copy link

Can one of the admins verify this patch?

@ligaz
Copy link

ligaz commented Aug 28, 2014

Can we omit the Telerik prefix from Telerik NativeScript CLI?

> On Windows systems, you can develop, build, and deploy NativeScript projects that target Android.

* Windows Vista or later
* Node.js 0.10.22 or later
Copy link

Choose a reason for hiding this comment

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

Should we provide links for those as well?

@ikoevska
Copy link
Contributor Author

@ligaz I have left Telerik in the first two instances of Telerik NativeScript and Telerik NativeScript CLI and removed it for the rest of the document.

However, my concern is that this is up to legal/marketing and their product name usages. There was a rule that within the context of *.telerik.com we can omit Telerik but we need to preserve it for all other cases. So, it might be best to consult with legal/marketing on this one.

@Fatme
Copy link
Contributor

Fatme commented Aug 29, 2014

👍

1 similar comment
@tytodorov
Copy link

👍

ikoevska added a commit that referenced this pull request Aug 29, 2014
@ikoevska ikoevska merged commit e296a62 into master Aug 29, 2014
@ikoevska ikoevska deleted the ns-readme branch August 29, 2014 11:17
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.

5 participants