Changeset 63422 for branches/7.0/tools/local-env/scripts/docker.js

Timestamp:

09/01/2026 05:50:10 PM (27 hours ago)

Author:

lancewillett

Message:

Build/Test Tools: Correct local environment failure handling.

Merges [62871] and [63416] to the 7.0 branch.

Developed in: ​https://github.com/WordPress/wordpress-develop/pull/13352

Reviewed by: johnbillion.
Props jonsurrell, lucasbustamante, mukesh27, adrianmoldovanwp.
See #65745.

Location:

branches/7.0

Files:

• . (modified) (1 prop)
• tools/local-env/scripts/docker.js (modified) (1 diff)

branches/7.0/tools/local-env/scripts/docker.js

@@ -39 +39 @@
 );
 
-process.exit( returns.status );
+if ( returns.error ) {
+        console.error( `Could not run Docker Compose. ${ returns.error.message }` );
+} else if ( returns.signal && returns.signal !== 'SIGINT' ) {
+        console.error( `Docker Compose was terminated by ${ returns.signal }.` );
+}
+
+// `status` is null when Docker could not be spawned at all, or was killed by a signal. SIGINT is
+// how a long-running command such as `env:logs` is normally ended, so it is not a failure worth an
+// npm error block. Every other signal means the command was killed before it finished.
+process.exit( returns.signal === 'SIGINT' ? 0 : ( returns.status ?? 1 ) );