potiuk commented on PR #58548:
URL: https://github.com/apache/airflow/pull/58548#issuecomment-3590659533
This is cool.
But there are two follow up questions:
* Currently `breeze testing ui-e2e-tests` fails when there is no pnpm
installed locally:
```
File
"/Users/jarekpotiuk/.local/share/uv/python/cpython-3.13.7-macos-aarch64-none/lib/python3.13/subprocess.py",
line 1972, in _execute_child
raise child_exception_type(errno_num, err_msg, err_filename)
FileNotFoundError: [Errno 2] No such file or directory: 'pnpm'
```
I think it should at least be explained as a prerequisite or installed
automatically. Generally `breeze testing` is a command that does **everything**
- lile building images when needed etc. Might be a good idea to improve that
part
* the tests are not run in CI (I guess?) - this means that they are prone
to be broken any time. Any plans to run them on CI ?
--
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]
