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

Update README to include sample setup detail #4

Closed
wants to merge 3 commits into from

Conversation

dwalkes
Copy link

@dwalkes dwalkes commented Oct 17, 2015

Thanks for sharing this SDK! I've added a few comments to the README based on some questions I had when using this for the first time today.

Add a bit more detail about sample setup steps and include links to relevant documentation.
Update README to include sample setup detail
Also update make argument example to select correct makefile with -f argument
@bhadrip
Copy link
Contributor

bhadrip commented Oct 23, 2015

Hello @dwalkes ,

Thanks for updating the steps.

Our process for handling these requests is to pull the changes into our internal system and push them to GitHub after following our testing and release process(Even though in this case it is documentation).

@chinawrj
Copy link

This is a very USEFUL guide. Why is it susppended for such a long time?

@chaurah
Copy link
Contributor

chaurah commented Jun 13, 2016

Hi @dwalkes and @chinawrj,
The Readme files were all updated for the v2.0.0 release of the SDK. The sample instructions are more detailed now, and we will keep refining them as we get more suggestions. As the Readme files are very different, I am closing this pull request. However, please do let us know if you have further suggestions on how to improve the Readme files we provide. We are constantly looking for ways to ensure people have a smooth getting started experience. Making sure the samples are easy to understand and run, is a very important part of that.

Rahul

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