Skip to content
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

Added instructions for CLI #57

Merged
merged 6 commits into from
Dec 30, 2017
Merged

Conversation

SubhrajitPrusty
Copy link
Contributor

No description provided.

@SubhrajitPrusty
Copy link
Contributor Author

fixes #55

@Parth-Vader
Copy link
Owner

@SubhrajitPrusty Great work, but can you add the contents of CLI.md in README.md itself? I don't think creating another file is required here.

@SubhrajitPrusty
Copy link
Contributor Author

It can be done. But I think it makes the README quite bulky, with the already present instructions and gifs.

@Parth-Vader
Copy link
Owner

It's fine I guess if there's an index and 'back to top' options. See this readme for example.

@SubhrajitPrusty
Copy link
Contributor Author

fixed.

@Parth-Vader
Copy link
Owner

@SubhrajitPrusty Great work! Thanks a lot. Merging!

@Parth-Vader Parth-Vader merged commit 616cfb7 into Parth-Vader:master Dec 30, 2017
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.

None yet

2 participants