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

doc: Improve examples and generate documentation pages with typedoc support #111

Draft
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

gosuri
Copy link
Member

@gosuri gosuri commented Dec 14, 2024

No description provided.

- Updated TypeScript to version 5.7.2 for improved features and performance.
- Ensured compatibility with existing TypeScript configuration.

Signed-off-by: Greg Osuri <[email protected]>
- Installed TypeDoc to facilitate automatic generation of project documentation.
- Updated package.json and package-lock.json to include TypeDoc as a development dependency.

Signed-off-by: Greg Osuri <[email protected]>
- Implemented `checkEndpointHealth` function to demonstrate how to retrieve and log the health status of network endpoints.
- Utilized `getEndpoints` and `getEndpointHealthStatus` to fetch and evaluate endpoint response times.
- Included error handling to manage potential issues during the health check process.

Signed-off-by: Greg Osuri <[email protected]>
@codecov-commenter
Copy link

codecov-commenter commented Dec 14, 2024

Codecov Report

Attention: Patch coverage is 0% with 25 lines in your changes missing coverage. Please review.

Project coverage is 69.02%. Comparing base (8be7bc2) to head (1d7d3e8).

Files with missing lines Patch % Lines
src/network/index.ts 0.00% 17 Missing ⚠️
src/wallet/index.ts 0.00% 7 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #111      +/-   ##
==========================================
- Coverage   70.13%   69.02%   -1.12%     
==========================================
  Files          23       23              
  Lines         864      878      +14     
  Branches      224      228       +4     
==========================================
  Hits          606      606              
- Misses        244      257      +13     
- Partials       14       15       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

- Added detailed description of the getAccount function.
- Included a @throws clause to document potential error conditions.
- Provided an example usage with error handling for better clarity.

Signed-off-by: Greg Osuri <[email protected]>
- Added comprehensive comments to all type definitions in `src/sdl/types.ts`.
- Improved clarity and understanding of v2 and v3 manifest structures.
- Documented optional parameters and resource configurations for better developer guidance.
- Ensured consistency in documentation style across all type definitions.

Signed-off-by: Greg Osuri <[email protected]>
- Generated updated documentation using Typedoc
- Ensured the latest changes are reflected in the docs
- Updated the docs script to open the documentation automatically

Signed-off-by: Greg Osuri <[email protected]>
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.

2 participants