-
Notifications
You must be signed in to change notification settings - Fork 389
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
docs: add gnoclient reference & how-to guide #1562
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #1562 +/- ##
==========================================
- Coverage 47.55% 44.83% -2.73%
==========================================
Files 388 459 +71
Lines 61279 67625 +6346
==========================================
+ Hits 29144 30322 +1178
- Misses 29695 34760 +5065
- Partials 2440 2543 +103 ☔ View full report in Codecov by Sentry. |
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.
Sounds good! Adding an example of a typical flow with gnoclient
would be cool.
Latest preview of the PR: https://www.loom.com/share/c51d5d18f0b74e6aa8e7aa388343e1bd?sid=0b12d2e3-2699-4d7a-b1d2-4dbb30deded1 |
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.
Looks good 💯
Description
This PR adds the reference documentation to the newly added gnoclient.
The reference docs for gnoclient are mostly generated by using a godoc to markdown tool, with slight modifications.
It also adds a how-to guide on how to connect to a Gno.land chain from Go.
Along the way:
tm2-js
&gno-js
gnoclient
function signatures to all be on the same type of receiver (all tests pass). I realize this could've been a different PR but its a small change with no functional/practical impact.Contributors' checklist...
BREAKING CHANGE: xxx
message was included in the description