-
Notifications
You must be signed in to change notification settings - Fork 948
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
Add Client Example In Readme File #2129
base: master
Are you sure you want to change the base?
Conversation
|
@RAJEEV2510 please move your sample to a seperate doc file under /Docs |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
see comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have create new sample client readme file
Hey @EthanChangAED Can you review my changes once again ,So anyone can easily use this |
Hi @RAJEEV2510 , thanks for your efforts. An example to use it you can find in some of the test code, e.g. server fixture |
@mregen Yes ,you are right, so u r going to merge it or you are going to close this issue . |
Proposed changes
I add the simple client example in readme file for better explations
Types of changes
What types of changes does your code introduce?
Put an
x
in the boxes that apply. You can also fill these out after creating the PR.Checklist
Put an
x
in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.Further comments
If this is a relatively large or complex change, kick off the discussion by explaining why you chose the solution you did and what alternatives you considered, etc...