Skip to content

Conversation

@rosen-vladimirov
Copy link
Contributor

In case user executes:

  • tns help - we should show the default index.html help with all commands
  • tns --help - we should show the default command line help generated from index.md file with all commands

This has been broken during changing html-help-service to help-service.
Fix the default behavior and add tests for it.

Merge after telerik/mobile-cli-lib#1024

Fix for #3204

In case user executes:
- `tns help` - we should show the default index.html help with all commands
- `tns --help` - we should show the default command line help generated from index.md file with all commands

This has been broken during changing html-help-service to help-service.
Fix the default behavior and add tests for it.
@rosen-vladimirov rosen-vladimirov force-pushed the vladimirov/fix-default-help branch from 4c307f5 to 54861b9 Compare November 9, 2017 09:04
@rosen-vladimirov
Copy link
Contributor Author

run ci

Copy link
Contributor

@Mitko-Kerezov Mitko-Kerezov left a comment

Choose a reason for hiding this comment

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

Looks okay to me

@rosen-vladimirov
Copy link
Contributor Author

run ci

@rosen-vladimirov
Copy link
Contributor Author

Merging as the failure is caused by infrastructure.

@rosen-vladimirov rosen-vladimirov merged commit e181f03 into master Nov 9, 2017
@rosen-vladimirov rosen-vladimirov deleted the vladimirov/fix-default-help branch November 9, 2017 11:57
rosen-vladimirov added a commit that referenced this pull request Nov 13, 2017
In case user executes:
- `tns help` - we should show the default index.html help with all commands
- `tns --help` - we should show the default command line help generated from index.md file with all commands

This has been broken during changing html-help-service to help-service.
Fix the default behavior and add tests for it.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants