-
Notifications
You must be signed in to change notification settings - Fork 77
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
doc: update xline client document #373
Conversation
Codecov ReportPatch coverage has no change and project coverage change:
Additional details and impacted files@@ Coverage Diff @@
## master #373 +/- ##
==========================================
- Coverage 52.36% 52.34% -0.03%
==========================================
Files 97 97
Lines 15160 15166 +6
Branches 15160 15166 +6
==========================================
Hits 7938 7938
- Misses 6728 6733 +5
- Partials 494 495 +1
☔ View full report in Codecov by Sentry. |
f129bee
to
7ed94f5
Compare
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.
api_type
, lease_gen
and AuthService
should not public to users
fixed, and I moved some of the re-exported api types to |
Signed-off-by: bsbds <[email protected]>
Signed-off-by: bsbds <[email protected]>
Signed-off-by: bsbds <[email protected]>
Signed-off-by: bsbds <[email protected]>
Signed-off-by: bsbds <[email protected]>
Signed-off-by: bsbds <[email protected]>
Signed-off-by: bsbds <[email protected]>
This PR is based on #372
Please briefly answer these questions:
what problem are you trying to solve? (or if there's no problem, what's the motivation for this change?)
descibed in
Update documents
section in [Checklist] Providing Xline client Rust SDK #369what changes does this pull request make?
Add more detailed descriptions to some methods and types.
are there any non-obvious implications of these changes? (does it break compatibility with previous versions, etc)