larvit/plainpages
3
0
Fork 0
Code Issues 1 Pull Requests 5 Actions 5 Packages Projects Releases Wiki Activity

Answer stability round 1: 0 🔴, 2 🟡 (0 misread); repoint the compose test's comment, split the :read guards' reasons

Browse Source
This commit is contained in:
Lilleman auf Larv
2026-11-08 21:09:19 +01:00
parent 7ad46d0adf
commit 4c8ba083ce
1 changed files with 2 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/compose.test.ts
+2 -1
View File
@@ -110,7 +110,8 @@ test("a one-shot bootstrap seeds the stack before web starts", () => {
}); });
test("deps live above WORKDIR, so no mount creates a root-owned dir in the checkout", () => { test("deps live above WORKDIR, so no mount creates a root-owned dir in the checkout", () => {
// A volume at /app/node_modules would leave a root-owned dir in the checkout (AGENTS.md). // A volume at /app/node_modules would leave a root-owned dir in the checkout
// (docs/decisions.md → "Dependencies install to `/node_modules`, above `WORKDIR /app`").
const dockerfile = read("Dockerfile"); const dockerfile = read("Dockerfile");
// split() returns the whole file when the marker is missing, widening "before" to "anywhere". // split() returns the whole file when the marker is missing, widening "before" to "anywhere".
assert.ok(dockerfile.includes("WORKDIR /app"), "the app dir is /app"); assert.ok(dockerfile.includes("WORKDIR /app"), "the app dir is /app");