Conversation
The build-and-push-app-image and build-and-push-webserver-image jobs always ran unconditionally, while build-and-push-cli-image was already conditional on build_cli_image. CLI-only repos without Dockerfile-app or Dockerfile-nginx fail on merge to main. Add build_app_image (default true) and build_webserver_image (default true) input flags with if: conditions, mirroring the existing build_cli_image pattern. Fully backward-compatible — existing callers that don't pass these inputs get the same behavior as before. Made-with: Cursor
PR SummaryLow Risk Overview Updates the Reviewed by Cursor Bugbot for commit f043fa0. Bugbot is set up for automated code reviews on this repo. Configure here. |
Summary
build_app_image(defaulttrue) andbuild_webserver_image(defaulttrue) input flags tophp-laravel-build-push.yamlif:conditions tobuild-and-push-app-imageandbuild-and-push-webserver-imagejobs, matching the existingbuild_cli_imagepatterntrue(unchanged behavior)Context
DEVEX-1204
CLI-only repos (like
marketplaces) don't haveDockerfile-apporDockerfile-nginx. The merge-to-main workflow fails withopen Dockerfile-app: no such file or directorybecause the app and webserver jobs always run unconditionally. The CLI job already had abuild_cli_imagetoggle — this PR adds the same pattern for the other two jobs.Test plan
listings) that don't pass the new inputs should build all three images as before (defaults aretrue)marketplaces) can passbuild_app_image: falseandbuild_webserver_image: falseto skip those jobsMade with Cursor