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

Generate markdown files for Solidity API documentation #133

Merged
merged 3 commits into from
Jan 4, 2024

Conversation

nkuba
Copy link
Member

@nkuba nkuba commented Jan 3, 2024

Use Open Zeppelin's solidity-docgen tool to generate markdown files for solidity contracts documentation.

To run the generation: pnpm run docs
The files will be output in gen/docs/ directory.

Use Open Zeppelin's [solidity-docgen
tool](https://github.com/OpenZeppelin/solidity-docgen) to generate
markdown files for solidity contracts documentation.

To run the generation: `pnpm run docs`
The files will be output in `gen/docs/` directory.
@nkuba nkuba requested review from dimpar and r-czajkowski January 3, 2024 22:51
@nkuba nkuba self-assigned this Jan 3, 2024
@dimpar
Copy link
Member

dimpar commented Jan 4, 2024

This is good. We should've added this at the beginning 😅

Can we go through the contracts and fix how the docs are being displayed? E.g. Acre.depositParameters docs display incorrectly.
Screenshot 2024-01-04 at 11 05 19

@nkuba
Copy link
Member Author

nkuba commented Jan 4, 2024

This is good. We should've added this at the beginning 😅

Can we go through the contracts and fix how the docs are being displayed? E.g. Acre.depositParameters docs display incorrectly. Screenshot 2024-01-04 at 11 05 19

Can we fix the contract documentation in another PR? I guess there will be quite a few things to fix :)

@dimpar dimpar enabled auto-merge January 4, 2024 11:15
@dimpar dimpar merged commit 4648f6b into main Jan 4, 2024
11 checks passed
@dimpar dimpar deleted the solidity-doc-gen branch January 4, 2024 12:26
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