Remove result code check

Former-commit-id: 6faa933d42

app/build_npms.py

@@ -85,18 +85,19 @@ def command(ctx, include, exclude):
                 print(f"Executing: {build_command}")
             envs = {"CERC_NPM_AUTH_TOKEN": os.environ["CERC_NPM_AUTH_TOKEN"]} | ({"CERC_SCRIPT_DEBUG": "true"} if debug else {})
             try:
-                build_result = docker.run("cerc/builder-js",
-                                          remove=True,
-                                          interactive=True,
-                                          tty=True,
-                                          user=f"{os.getuid()}:{os.getgid()}",
-                                          envs=envs,
-                                          add_hosts=[("gitea.local", "host-gateway")],
-                                          volumes=[(repo_full_path, "/workspace")],
-                                          command=build_command
-                                          )
-                # TODO: check result in build_result.returncode
-                print(f"Result is: {build_result}")
+                docker.run("cerc/builder-js",
+                           remove=True,
+                           interactive=True,
+                           tty=True,
+                           user=f"{os.getuid()}:{os.getgid()}",
+                           envs=envs,
+                           add_hosts=[("gitea.local", "host-gateway")],
+                           volumes=[(repo_full_path, "/workspace")],
+                           command=build_command
+                           )
+                # Note that although the docs say that build_result should contain
+                # the command output as a string, in reality it is always the empty string.
+                # Since we detect errors via catching exceptions below, we can safely ignore it here.
             except DockerException as e:
                 print(f"FATAL error executing build in container:\n {e}")
         else: