Skip to content

Commit 1f76794

Browse files
authored
Merge pull request #256 from platformsh/require-start-stack-php
chore: add test case for pre_start on composable
2 parents bac768b + 611f47d commit 1f76794

File tree

2 files changed

+9
-3
lines changed

2 files changed

+9
-3
lines changed

validator/schema/upsun.json

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -247,11 +247,13 @@
247247
"type": "string",
248248
"title": "The command used to start the application",
249249
"description": "It will be restarted if it terminates. Do not use on PHP unless using a custom persistent process like React PHP or FrankenPHP. \nSee note: \nhttps://docs.upsun.com/anchors/app/reference/web/commands/start/"
250+
},
251+
"post_start": {
252+
"type": "string",
253+
"title": "A command executed after the application is started",
254+
"description": "More information: \nhttps://docs.upsun.com/anchors/app/reference/web/commands/"
250255
}
251256
},
252-
"required": [
253-
"start"
254-
],
255257
"additionalProperties": false,
256258
"title": "The command to launch your app",
257259
"description": "More information: \nhttps://docs.upsun.com/anchors/app/reference/web/commands/"

validator/validator_test.go

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -147,6 +147,10 @@ applications:
147147
- pdo_sqlite
148148
- "nodejs@20"
149149
150+
web:
151+
commands:
152+
pre_start: echo "pre_start"
153+
post_start: echo "post_start"
150154
`,
151155
),
152156
},

0 commit comments

Comments
 (0)