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

more typedoc #8720

Merged
merged 7 commits into from
Jan 17, 2024
Merged

more typedoc #8720

merged 7 commits into from
Jan 17, 2024

Conversation

turadg
Copy link
Member

@turadg turadg commented Jan 8, 2024

Description

Include several more packages in the TypeDoc build

Security Considerations

Scaling Considerations

Documentation Considerations

Testing Considerations

Upgrade Considerations

@turadg turadg requested a review from dckc January 8, 2024 19:26
Copy link
Member

@dckc dckc left a comment

Choose a reason for hiding this comment

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

This has me wishing for deploy previews.

@@ -0,0 +1,3 @@
# Vats

Builders for Vats on the Agoric chain that aren't contracts.
Copy link
Member

Choose a reason for hiding this comment

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

there are some contracts here too; mintHolder and centralSupply come to mind.
But this will do for now.

"src/offers.js",
"src/smartWallet.js",
"src/types.js",
"src/walletFactory.js",
Copy link
Member

Choose a reason for hiding this comment

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

walletFactory doesn't subsume the others as entry points?

(interesting that we don't have a "main" key in smart-wallet/package.json)

Copy link
Member Author

Choose a reason for hiding this comment

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

Nope. I added these after getting warnings like,

[warning] InvitationSpec, defined in ./packages/smart-wallet/src/invitations.js, is referenced by offers.OfferSpec.__type.invitationSpec but not included in the documentation.
[warning] Petname, defined in ./packages/smart-wallet/src/types.d.ts, is referenced by smartWallet.BrandDescriptor.__type.petname but not included in the documentation.
[warning] MakeOfferWatcher, defined in ./packages/smart-wallet/src/offerWatcher.js, is referenced by smartWallet.ExecutorPowers.__type.makeOfferWatcher but not included in the documentation.
[warning] BrandDescriptor, defined in ./packages/smart-wallet/src/walletFactory.js, is referenced by walletFactory.makeAssetRegistry.makeAssetRegistry.__type.get.__type.__type but not included in the documentation.

@turadg turadg added the automerge:rebase Automatically rebase updates, then merge label Jan 17, 2024
@mergify mergify bot merged commit 51f64c2 into master Jan 17, 2024
66 checks passed
@mergify mergify bot deleted the ta/typedoc-vats branch January 17, 2024 18:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automerge:rebase Automatically rebase updates, then merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants