potiuk commented on code in PR #58845:
URL: https://github.com/apache/airflow/pull/58845#discussion_r2573154752


##########
dev/breeze/src/airflow_breeze/utils/run_utils.py:
##########
@@ -260,6 +260,42 @@ def assert_prek_installed():
         sys.exit(1)
 
 
+def check_pnpm_installed():

Review Comment:
   Actually what we do in other cases, we are using `prek` node hook. Prek 
already has built-in node installer and it can have "pnpm" as additional 
dependency, it also handles specification of which version of node to install 
etc.  so we can likely do similar thing? 



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to