Skip to content

Commit

Permalink
handle TODOs
Browse files Browse the repository at this point in the history
  • Loading branch information
dr-bonez committed Dec 2, 2024
1 parent e6fbaf6 commit dc563d1
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 6 deletions.
5 changes: 4 additions & 1 deletion sdk/package/lib/mainFn/Daemons.ts
Original file line number Diff line number Diff line change
Expand Up @@ -53,7 +53,10 @@ type DaemonsParams<
image: {
/** The ID of the image. Must be one of the image IDs declared in the manifest */
id: keyof Manifest["images"] & T.ImageId
/** @TODO Bonez */
/**
* Whether or not to share the `/run` directory with the parent container.
* This is useful if you are trying to connect to a service that exposes a unix domain socket or auth cookie via the `/run` directory
*/
sharedRun?: boolean
}
/** For mounting the necessary volumes. Syntax: sdk.Mounts.of().addVolume() */
Expand Down
10 changes: 5 additions & 5 deletions sdk/package/lib/mainFn/Mounts.ts
Original file line number Diff line number Diff line change
Expand Up @@ -30,9 +30,9 @@ export class Mounts<Manifest extends T.SDKManifest> {
}

addVolume(
/** The ID of the volume to mount. Much be one of the volume IDs defined in the manifest */
/** The ID of the volume to mount. Must be one of the volume IDs defined in the manifest */
id: Manifest["volumes"][number],
/** @TODO Bonez */
/** The path within the volume to mount. Use `null` to mount the entire volume */
subpath: string | null,
/** Where to mount the volume. e.g. /data */
mountpoint: string,
Expand All @@ -49,9 +49,9 @@ export class Mounts<Manifest extends T.SDKManifest> {
}

addAssets(
/** @TODO Bonez. I thought the assets array in the manifest was an array of path to asset directories. They have IDs? */
/** The ID of the asset directory to mount. This is typically the same as the folder name in your assets directory */
id: Manifest["assets"][number],
/** @TODO Bonez */
/** The path within the asset directory to mount. Use `null` to mount the entire volume */
subpath: string | null,
/** Where to mount the asset. e.g. /asset */
mountpoint: string,
Expand All @@ -69,7 +69,7 @@ export class Mounts<Manifest extends T.SDKManifest> {
dependencyId: keyof Manifest["dependencies"] & string,
/** The ID of the volume belonging to the dependency service to mount */
volumeId: DependencyManifest["volumes"][number],
/** @TODO Bonez */
/** The path within the dependency's volume to mount. Use `null` to mount the entire volume */
subpath: string | null,
/** Where to mount the dependency's volume. e.g. /service-id */
mountpoint: string,
Expand Down

0 comments on commit dc563d1

Please sign in to comment.