-
Notifications
You must be signed in to change notification settings - Fork 27
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
contracts: Add solidity API reference #187
Conversation
To reduce the size of the documentation many of the 'internal' items can be safely made 'private' without changing the consumable API. I tried the multi-page I found another documentation generator called Dodoc which has a better structure, e.g.: https://library.primitive.xyz/technical/smart-contracts/autogenerated-docs/manager/base/CashManager I suggest removing tests from the docs: diff --git a/contracts/hardhat.config.ts b/contracts/hardhat.config.ts
index 6bc37c2..94bb7c3 100644
--- a/contracts/hardhat.config.ts
+++ b/contracts/hardhat.config.ts
@@ -105,6 +105,14 @@ const config: HardhatUserConfig = {
},
docgen: {
pages: 'single',
+ exclude: [
+ 'tests/SigningTests.sol',
+ 'tests/SubcallTests.sol',
+ 'tests/SignedQueriesTests.sol',
+ 'tests/EIP155Tests.sol',
+ 'tests/HashTests.sol',
+ 'OPL.sol'
+ ]
},
}; |
d23eb8a
to
40afd0b
Compare
c1d7cdb
to
9cd36be
Compare
Is there a reason to ignore |
a4ad732
to
a6ac267
Compare
fa0d3c6
to
dc6685f
Compare
OPL.sol is just a meta package https://github.com/oasisprotocol/sapphire-paratime/blob/main/contracts/contracts/OPL.sol |
26d18d2
to
6c1d4cd
Compare
6c1d4cd
to
2204d5e
Compare
…atevz/feature/solidity-docgen contracts: Add solidity API reference bc27209
…tevz/feature/solidity-docgen contracts: Add solidity API reference bc27209
Adds Solidity API reference using
forge doc --build
to generate the markdowns and final HTMLs.Depends on #186.
Preview