-
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
export withOrchestration #9808
export withOrchestration #9808
Changes from all commits
c388feb
c3a6fa3
f924f60
3417099
d343ba1
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 |
---|---|---|
|
@@ -20,7 +20,7 @@ import { makeZoeTools } from './zoe-tools.js'; | |
* @import {NameHub} from '@agoric/vats'; | ||
* @import {Remote} from '@agoric/vow'; | ||
* @import {Zone} from '@agoric/zone'; | ||
* @import {CosmosInterchainService} from '../exos/cosmos-interchain-service.js'; | ||
* @import {CosmosInterchainService} from '../types.js'; | ||
*/ | ||
|
||
/** | ||
|
@@ -43,6 +43,7 @@ import { makeZoeTools } from './zoe-tools.js'; | |
* @param {Baggage} baggage | ||
* @param {OrchestrationPowers} remotePowers | ||
* @param {Marshaller} marshaller | ||
* @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. doesn't seem to have the desired effect. I still see it: https://ta-export-withorchestration.agoric-sdk.pages.dev/modules/_agoric_orchestration 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. Good catch. It needs the excludeInternal option, but even with that it's appearing. I'll wait for #9727 and try again. I don't want to create merge conflicts for that one and maybe it fixes the bug. 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 tried merging and that didn't fix it. Now I suspect a bug in typedoc 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. yeah, I've tried without luck to get |
||
*/ | ||
export const provideOrchestration = ( | ||
zcf, | ||
|
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.
this
getPower
thing still isn't gone? darn.