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

Replace README API docs with generated docs #213

Merged
merged 3 commits into from
Sep 21, 2021
Merged

Conversation

Gudahtt
Copy link
Member

@Gudahtt Gudahtt commented Sep 21, 2021

The API documentation in the README has been replaced with GitHub-pages hosted documentation generated using TypeDoc.

The @MetaMask/action-publish-gh-pages action is used for the publish step. The action configuration is taken nearly exactly from the README example for that action, except that the build command and directory are updated, and the yarn setup step has been added (because TypeDoc needs to be installed).

A new docs:publish npm script has been added to ensure the output directory doesn't get cleaned (because it includes the git worktree setup by the @MetaMask/action-publish-gh-pages action), and to set the correct git revision.

These docs will only be published when the package is released, rather than the latest development version. Instructions have been added to the contributing section showing how to generate and view the API documentation for the current commit.

The old README API documentation has been reviewed and incorporated into the inline documentation. Minor adjustments and improvements have been made to ensure that no information is lost.

Closes #47

The API documentation in the README has been replaced with GitHub-pages
hosted documentation generated using TypeDoc.

The `@MetaMask/action-publish-gh-pages` action is used for the publish
step. The action configuration is taken nearly exactly from the README
example for that action, except that the build command and directory
are updated, and the `yarn setup` step has been added (because TypeDoc
needs to be installed).

A new `docs:publish` npm script has been added to ensure the output
directory doesn't get cleaned (because it includes the `git worktree`
setup by the `@MetaMask/action-publish-gh-pages` action), and to set
the correct git revision.

These docs will only be published when the package is released, rather
than the latest development version. Instructions have been added to
the contributing section showing how to generate and view the API
documentation for the current commit.

The old README API documentation has been reviewed and incorporated
into the inline documentation. Minor adjustments and improvements have
been made to ensure that no information is lost.
@Gudahtt Gudahtt requested a review from a team as a code owner September 21, 2021 14:44
@Gudahtt
Copy link
Member Author

Gudahtt commented Sep 21, 2021

I deployed and tested this on my fork, which you can see here: https://gudahtt.github.io/eth-sig-util/index.html

Copy link
Member

@rekmarks rekmarks left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Very nice!

@Gudahtt Gudahtt merged commit 8f5a90b into main Sep 21, 2021
@Gudahtt Gudahtt deleted the replace-README-API-docs branch September 21, 2021 21:24
rekmarks added a commit to MetaMask/utils that referenced this pull request May 15, 2022
Adds automated API docs generation via `typedoc` and automated publishing to GitHub pages. Closely follows the implementation in MetaMask/eth-sig-util#207 and MetaMask/eth-sig-util#213, but uses https://github.com/peaceiris/actions-gh-pages instead of our own GitHub Pages action, which we plan to sunset.
mcmire pushed a commit to mcmire/core that referenced this pull request Jul 17, 2023
Adds automated API docs generation via `typedoc` and automated publishing to GitHub pages. Closely follows the implementation in MetaMask/eth-sig-util#207 and MetaMask/eth-sig-util#213, but uses https://github.com/peaceiris/actions-gh-pages instead of our own GitHub Pages action, which we plan to sunset.
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.

readme methods section is outdated
2 participants