Merge pull request #1486 from revant/fix-verbose

fix: pass verbose to yarn or npm install

bench/app.py

@@ -578,7 +578,8 @@ def install_app(
 		install_python_dev_dependencies(apps=app, bench_path=bench_path, verbose=verbose)
 
 	if os.path.exists(os.path.join(app_path, "package.json")):
-		bench.run("yarn install", cwd=app_path)
+		yarn_install = "yarn install --verbose" if verbose else "yarn install"
+		bench.run(yarn_install, cwd=app_path)
 
 	bench.apps.sync(app_name=app, required=resolution, branch=tag, app_dir=app_path)
 

bench/utils/bench.py

@@ -47,7 +47,7 @@ def get_venv_path(verbose=False, python="python3"):
 		log("venv cannot be found", level=2)
 
 
-def update_node_packages(bench_path=".", apps=None):
+def update_node_packages(bench_path=".", apps=None, verbose=None):
 	print("Updating node packages...")
 	from distutils.version import LooseVersion
 
@@ -58,9 +58,9 @@ def update_node_packages(bench_path=".", apps=None):
 	# After rollup was merged, frappe_version = 10.1
 	# if develop_verion is 11 and up, only then install yarn
 	if v < LooseVersion("11.x.x-develop"):
-		update_npm_packages(bench_path, apps=apps)
+		update_npm_packages(bench_path, apps=apps, verbose=verbose)
 	else:
-		update_yarn_packages(bench_path, apps=apps)
+		update_yarn_packages(bench_path, apps=apps, verbose=verbose)
 
 
 def install_python_dev_dependencies(bench_path=".", apps=None, verbose=False):
@@ -110,9 +110,11 @@ def _generate_dev_deps_pattern(pyproject_path):
 	return requirements_pattern
 
 
-def update_yarn_packages(bench_path=".", apps=None):
+def update_yarn_packages(bench_path=".", apps=None, verbose=None):
+	import bench.cli as bench_cli
 	from bench.bench import Bench
 
+	verbose = bench_cli.verbose or verbose
 	bench = Bench(bench_path)
 	apps = apps or bench.apps
 	apps_dir = os.path.join(bench.name, "apps")
@@ -127,10 +129,13 @@ def update_yarn_packages(bench_path=".", apps=None):
 		app_path = os.path.join(apps_dir, app)
 		if os.path.exists(os.path.join(app_path, "package.json")):
 			click.secho(f"\nInstalling node dependencies for {app}", fg="yellow")
-			bench.run("yarn install", cwd=app_path)
+			yarn_install = "yarn install --verbose" if verbose else "yarn install"
+			bench.run(yarn_install, cwd=app_path)
 
 
-def update_npm_packages(bench_path=".", apps=None):
+def update_npm_packages(bench_path=".", apps=None, verbose=None):
+	verbose = bench.cli.verbose or verbose
+	npm_install = "npm install --verbose" if verbose else "npm install"
 	apps_dir = os.path.join(bench_path, "apps")
 	package_json = {}
 
@@ -162,7 +167,7 @@ def update_npm_packages(bench_path=".", apps=None):
 	with open(os.path.join(bench_path, "package.json"), "w") as f:
 		f.write(json.dumps(package_json, indent=1, sort_keys=True))
 
-	exec_cmd("npm install", cwd=bench_path)
+	exec_cmd(npm_install, cwd=bench_path)
 
 
 def migrate_env(python, backup=False):