Skip to content
This repository has been archived by the owner on Jan 13, 2025. It is now read-only.

Move README content into TypeDoc-compatible code comments #3577

Closed
steveluscher opened this issue Nov 15, 2024 · 1 comment · Fixed by #3601
Closed

Move README content into TypeDoc-compatible code comments #3577

steveluscher opened this issue Nov 15, 2024 · 1 comment · Fixed by #3601
Labels
enhancement New feature or request

Comments

@steveluscher
Copy link
Contributor

steveluscher commented Nov 15, 2024

Motivation

If the developer is unaware of the presence of the README files in each subpackage, or has not yet found the READMEs rendered at https://solana-labs.github.io/solana-web3.js, they may have questions that are answered by those READMEs.

Let's move (or perhaps copy) all of the content possible into the code itself, in a format that's compatible with TypeDoc. This will bring all of that knowledge into developers' IDE, on hover et al.

Details

  • Read the TypeDoc specification (link)
    • Get really familiar with all of the tags available, and how you would use them to express the documentation
    • Become expert at creating declaration references, so that all of the documentation autolinks to all of the rest.
  • Constantly test the docs rendered by your PR in VS Code. Supply screenshots when appropriate
  • Constantly test the HTML docs rendered by your PR using pnpm compile:docs
  • Make many of the smallest PR you can, so that each change can be reviewed easily
@steveluscher steveluscher added the enhancement New feature or request label Nov 15, 2024
@steveluscher steveluscher linked a pull request Nov 25, 2024 that will close this issue
Copy link
Contributor

github-actions bot commented Dec 2, 2024

Because there has been no activity on this issue for 7 days since it was closed, it has been automatically locked. Please open a new issue if it requires a follow up.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Dec 2, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant