The GitHub Actions job "Tests" on airflow.git has failed.
Run started by GitHub user potiuk (triggered by potiuk).

Head commit for run:
11c46fd2ec165da32202b464be7c2df5cca4d6c0 / Jarek Potiuk <[email protected]>
Upgrade to latest ruff and remove ISC001 warning from output (#36649)

This PR upgrades to latest ruff, and removes the ISC001 warning that
warns us against potential conflict between ruff and ruff-formatter
when two strings in one line get concatenated.

This warning makes sense if you run both ruff and formatting at the
same time, but in our case we are doing it in two separate
steps - one step is to run ruff linting and the second step is to
run formatting and running formatting already runs after linting
is complete.

This warnign is pretty misleading as it distracts from real formatting
issues you might have.

There is - unfortunately - no standard way to remove the warning
so we have to do it a little "around" - rather than running
the pre-commit directly from ruff website, we run our local pre-commit
with few lines of Python code that runs ruff through shell and
greps out the ISC001 warning. We also force color to make
sure the output is still coloured.

Report URL: https://github.com/apache/airflow/actions/runs/7440141422

With regards,
GitHub Actions via GitBox


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to