johnslavik commented on PR #58952:
URL: https://github.com/apache/airflow/pull/58952#issuecomment-3602900675
When the prek version is lower than required, the error message is now much
more helpful:
```
mpleAuthManager detected. All roles (admin, viewer, user, op) are always
available via configuration in
.dev/breeze/src/airflow_breeze/files/simple_auth_manager_passwords.json
The asset compilation failed. Exiting.
error: Failed to parse `.pre-commit-config.yaml`
caused by: Required minimum prek version `0.2.0` is greater than current
version `0.1.1`. Please consider updating prek. at line 18 column 1
Error 1 returned
```
(previously it was just `No hook found with id `compile-ui-assets` in stage
manual`)
Maybe in a followup PR we could make `breeze start-airflow` fail early
though?
In both scenarios the script assumes it's waiting for a job that started
successfully and could fail, not a for a job that didn't start.
--
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]
