Stuck on an issue?

Lightrun Answers was designed to reduce the constant googling that comes with debugging 3rd party libraries. It collects links to all the places you might be looking at while hunting down a tough bug.

And, if you’re still stuck at the end, we’re happy to hop on a call to see how we can help out.

Breeze: Add nicer error for BREEZE.rst update static check

See original GitHub issue

If you modify BREEZE.rst or any of the “dev” files there is a static check run that updates screenshots in the docs.

If you have no Breeze installed, It should nicely inform the user that they have to install Breeze and redirect them to the documentation on how to do it.

But currently, it will fail with this message:

Update output of breeze commands in BREEZE.rst...........................................Failed
- hook id: update-breeze-file
- exit code: 1

Traceback (most recent call last):
  File "./scripts/ci/pre_commit/pre_commit_breeze_cmd_line.py", line 81, in <module>
    print_help_for_all_commands()
  File "./scripts/ci/pre_commit/pre_commit_breeze_cmd_line.py", line 46, in print_help_for_all_commands
    check_call(["breeze", "--help"], env=env)
  File "/opt/hostedtoolcache/Python/3.7.12/x64/lib/python3.7/subprocess.py", line 358, in check_call
    retcode = call(*popenargs, **kwargs)
  File "/opt/hostedtoolcache/Python/3.7.12/x64/lib/python3.7/subprocess.py", line 339, in call
    with Popen(*popenargs, **kwargs) as p:
  File "/opt/hostedtoolcache/Python/3.7.12/x64/lib/python3.7/subprocess.py", line 800, in __init__
    restore_signals, start_new_session)
  File "/opt/hostedtoolcache/Python/3.7.12/x64/lib/python3.7/subprocess.py", line 1551, in _execute_child
    raise child_exception_type(errno_num, err_msg, err_filename)
FileNotFoundError: [Errno 2] No such file or directory: 'breeze': 'breeze'

This should be fixed.

Issue Analytics

State:
Created a year ago
Comments:7 (7 by maintainers)

Top GitHub Comments

1reaction

potiukcommented, May 8, 2022

Actually I found a way to run breeze in pre-commit even if you DO NOT have it installed. Since we are using it in a very limited way, we just need “rich” and “rich-click” addded in the .pre-commit environment, adding a PATH to breeze and we should be able to run breeze to get --help output as well as to check which configuration is available. Fix is coming in #23570

1reaction

potiukcommented, Apr 12, 2022

Not for the message 😃

Top Results From Across the Web

[GitHub] [airflow] potiuk opened a new issue, #22908: Add nicer ...

If you modify BREEZE.rst or any of the "dev" files there is a static check run that updates screenshots in the docs. If...

BREEZE.rst - airflow - Git at Google

Run static checks - either for currently staged change or for all files with. ``breeze static-check`` command. Build documentation:.

Changelog — Airflow Documentation

Add airflow jobs check CLI command to check health of jobs (Scheduler etc) (#14519) ... Get rid of Airflow 1.10 in Breeze (#15712)....

Xso - River Thames Conditions - Environment Agency - GOV.UK

... Centro medico sawaya feira de santana, Error analysis table, Breeze backup, Rinconcito super latino cutler bay menu, Toolcat reviews, Photo cards insert...

CONTRIBUTING.rst ... - GitLab

Documentation; Static code checks; Coding style and best practices ... All details about using and running Airflow Breeze can be found in BREEZE.rst....