-
Notifications
You must be signed in to change notification settings - Fork 212
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
orch ref docs fixes (WIP) #9768
base: master
Are you sure you want to change the base?
Changes from all commits
e44fe11
98e3749
8551557
1cdea29
d5aceb3
b78fbfa
3a20047
287fd21
bac6e4d
616cbee
71291c9
af4f187
f12c8f6
cb66f49
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -171,15 +171,16 @@ | |
}); | ||
|
||
/** | ||
* Make a new ChainHub in the zone (or in the heap if no zone is provided). | ||
* Make a new ChainHub in the heap zone. | ||
* | ||
* The resulting object is an Exo singleton. It has no precious state. It's only | ||
* The resulting object is an Exo singleton. It has no precious state. Its only | ||
* state is a cache of queries to agoricNames and whatever info was provided in | ||
* registration calls. When you need a newer version you can simply make a hub | ||
* hub and repeat the registrations. | ||
* registration calls. When you need a newer version you can simply make a ChainHub | ||
* and repeat the registrations. | ||
* | ||
* @param {Remote<NameHub>} agoricNames | ||
* @param {VowTools} vowTools | ||
* @internal @see {withOrchestration} | ||
Check warning on line 183 in packages/orchestration/src/exos/chain-hub.js GitHub Actions / lint-rest
|
||
*/ | ||
export const makeChainHub = (agoricNames, vowTools) => { | ||
const zone = makeHeapZone(); | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -3,11 +3,21 @@ import { VowShape } from '@agoric/vow'; | |
import { M } from '@endo/patterns'; | ||
|
||
/** | ||
* @import {TypedPattern} from '@agoric/internal'; | ||
* @import {TypedPattern as TPInternal} from '@agoric/internal'; | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. why rename this? If There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. The problem was: The docs showed So this is an alias in the orchestration so that it gets linked. For example, see: Moving There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. can we leave it unlinked? if There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
I'm making a separate issue about it; we can decide to close it without action: |
||
* @import {ChainAddress, CosmosAssetInfo, ChainInfo, CosmosChainInfo, DenomAmount, DenomDetail} from './types.js'; | ||
* @import {Delegation} from '@agoric/cosmic-proto/cosmos/staking/v1beta1/staking.js'; | ||
*/ | ||
|
||
/** | ||
* @template T | ||
* @typedef {TPInternal<T>} TypedPattern a pattern that recognizes an object | ||
* with the static type T | ||
* @see {M} | ||
* | ||
* TODO: caveat about static types that express more than patterns can. | ||
* TODO: push TypedPattern down into @endo/patterns | ||
*/ | ||
|
||
/** | ||
* Used for IBC Channel Connections that only send outgoing transactions. If | ||
* your channel expects incoming transactions, please extend this interface to | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
https://typedoc.org/tags/private/ says it shouldn't be used. I think we want @internal
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've struggled to get
@internal
to work. @mitdralla suggested@private
, so I tried that. Also didn't work.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For now I think we should assume we can get it working. Even when it's not omitted it has an INTERNAL tag which is better than nothing. #9808 (comment)