Skip to content

Commit 71d280d

Browse files
committed
ci: fix container build script
Invoke docker build with Bun shell so commands run correctly, and document default automation behavior.
1 parent 5cfb5fd commit 71d280d

File tree

2 files changed

+7
-3
lines changed

2 files changed

+7
-3
lines changed

AGENTS.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
- To regenerate the JavaScript SDK, run `./packages/sdk/js/script/build.ts`.
22
- ALWAYS USE PARALLEL TOOLS WHEN APPLICABLE.
33
- The default branch in this repo is `dev`.
4+
- Prefer automation: execute requested actions without confirmation unless blocked by missing info or safety/irreversibility.
45

56
## Style Guide
67

packages/containers/script/build.ts

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -15,9 +15,12 @@ for (const name of images) {
1515
const image = `${reg}/build/${name}:${tag}`
1616
const file = `packages/containers/${name}/Dockerfile`
1717
const arg = name === "base" ? "" : `--build-arg REGISTRY=${reg}`
18-
const cmd = `docker build -f ${file} -t ${image} ${arg} .`
19-
console.log(cmd)
20-
await $`${cmd}`
18+
console.log(`docker build -f ${file} -t ${image} ${arg} .`)
19+
if (arg) {
20+
await $`docker build -f ${file} -t ${image} --build-arg REGISTRY=${reg} .`
21+
} else {
22+
await $`docker build -f ${file} -t ${image} .`
23+
}
2124

2225
if (push) {
2326
await $`docker push ${image}`

0 commit comments

Comments
 (0)