Re: [PR] Update BREEZE.rst with different test example [airflow]

2023-12-27 Thread via GitHub
potiuk merged PR #36234: URL: https://github.com/apache/airflow/pull/36234 -- 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: commits-unsubscr...@airflow.a

Re: [PR] Update BREEZE.rst with different test example [airflow]

2023-12-16 Thread via GitHub
potiuk commented on PR #36234: URL: https://github.com/apache/airflow/pull/36234#issuecomment-1858931573 This code is wrong likely - it should be exit(0) ``` if os.environ.get("SKIP_BREEZE_PRE_COMMITS"): console.print("[yellow]Skipping breeze pre-commit as SKIP_BREEZ

Re: [PR] Update BREEZE.rst with different test example [airflow]

2023-12-15 Thread via GitHub
Bowrna commented on PR #36234: URL: https://github.com/apache/airflow/pull/36234#issuecomment-1857449237 static checks are failing with following message `Skipping breeze pre-commit as SKIP_BREEZE_PRE_COMMIT is set`. However it works fine when I run the breeze static-checks --only-my-change

Re: [PR] Update BREEZE.rst with different test example [airflow]

2023-12-14 Thread via GitHub
Bowrna closed pull request #36234: Update BREEZE.rst with different test example URL: https://github.com/apache/airflow/pull/36234 -- 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

[PR] Update BREEZE.rst with different test example [airflow]

2023-12-14 Thread via GitHub
Bowrna opened a new pull request, #36234: URL: https://github.com/apache/airflow/pull/36234 Updating the breeze docs with different pytest example as the function mentioned in the example is removed from the test_core.py --- **^ Add meaningful descriptio