amoghrajesh commented on code in PR #51603: URL: https://github.com/apache/airflow/pull/51603#discussion_r2139262447
########## airflow-core/docs/installation/upgrading_to_airflow3.rst: ########## @@ -46,31 +46,40 @@ Step 2: Clean and back up your existing Airflow Instance ensure you deploy your changes to your old instance prior to upgrade, and wait until your dags have all been reprocessed (and all errors gone) before you proceed with upgrade. -Step 3: DAG Authors - Check your Airflow DAGs for compatibility +Step 3: Dag Authors - Check your Airflow dags for compatibility Review Comment: Hmmm -- not so sure about this one! ########## airflow-core/docs/installation/upgrading_to_airflow3.rst: ########## @@ -46,31 +46,40 @@ Step 2: Clean and back up your existing Airflow Instance ensure you deploy your changes to your old instance prior to upgrade, and wait until your dags have all been reprocessed (and all errors gone) before you proceed with upgrade. -Step 3: DAG Authors - Check your Airflow DAGs for compatibility +Step 3: Dag Authors - Check your Airflow dags for compatibility ---------------------------------------------------------------- -To minimize friction for users upgrading from prior versions of Airflow, we have created a dag upgrade check utility using `Ruff <https://docs.astral.sh/ruff/>`_. +To minimize friction for users upgrading from prior versions of Airflow, we have created a dag upgrade check utility using `Ruff <https://docs.astral.sh/ruff/>`_ combined with `AIR <https://docs.astral.sh/ruff/rules/#airflow-air>`_ rules. The rules AIR301 and AIR302 indicate breaking changes in Airflow 3, while AIR311 and AIR312 highlight changes that are not currently breaking but are strongly recommended for updates. -The latest available ``ruff`` version will have the most up-to-date rules, but be sure to use at least version ``0.11.6``. The below example demonstrates how to check -for dag incompatibilities that will need to be fixed before they will work as expected on Airflow 3. +The latest available ``ruff`` version will have the most up-to-date rules, but be sure to use at least version ``0.11.13``. The below example demonstrates how to check for dag incompatibilities that will need to be fixed before they will work as expected on Airflow 3. Review Comment: ```suggestion The latest available ``ruff`` version will have the most up-to-date rules, but be sure to use at least version ``0.11.13``. The below example demonstrates how to check for dag incompatibilities that will need to be fixed before they will work as expected on Airflow 3. ``` ########## airflow-core/docs/installation/upgrading_to_airflow3.rst: ########## @@ -46,31 +46,40 @@ Step 2: Clean and back up your existing Airflow Instance ensure you deploy your changes to your old instance prior to upgrade, and wait until your dags have all been reprocessed (and all errors gone) before you proceed with upgrade. -Step 3: DAG Authors - Check your Airflow DAGs for compatibility +Step 3: Dag Authors - Check your Airflow dags for compatibility ---------------------------------------------------------------- -To minimize friction for users upgrading from prior versions of Airflow, we have created a dag upgrade check utility using `Ruff <https://docs.astral.sh/ruff/>`_. +To minimize friction for users upgrading from prior versions of Airflow, we have created a dag upgrade check utility using `Ruff <https://docs.astral.sh/ruff/>`_ combined with `AIR <https://docs.astral.sh/ruff/rules/#airflow-air>`_ rules. The rules AIR301 and AIR302 indicate breaking changes in Airflow 3, while AIR311 and AIR312 highlight changes that are not currently breaking but are strongly recommended for updates. Review Comment: ```suggestion To minimize friction for users upgrading from prior versions of Airflow, we have created a dag upgrade check utility using `Ruff <https://docs.astral.sh/ruff/>`_ combined with `AIR <https://docs.astral.sh/ruff/rules/#airflow-air>`_ rules. The rules AIR301 and AIR302 indicate breaking changes in Airflow 3, while AIR311 and AIR312 highlight changes that are not currently breaking but are strongly recommended for updates. ``` ########## airflow-core/docs/installation/upgrading_to_airflow3.rst: ########## @@ -46,31 +46,40 @@ Step 2: Clean and back up your existing Airflow Instance ensure you deploy your changes to your old instance prior to upgrade, and wait until your dags have all been reprocessed (and all errors gone) before you proceed with upgrade. -Step 3: DAG Authors - Check your Airflow DAGs for compatibility +Step 3: Dag Authors - Check your Airflow dags for compatibility ---------------------------------------------------------------- -To minimize friction for users upgrading from prior versions of Airflow, we have created a dag upgrade check utility using `Ruff <https://docs.astral.sh/ruff/>`_. +To minimize friction for users upgrading from prior versions of Airflow, we have created a dag upgrade check utility using `Ruff <https://docs.astral.sh/ruff/>`_ combined with `AIR <https://docs.astral.sh/ruff/rules/#airflow-air>`_ rules. The rules AIR301 and AIR302 indicate breaking changes in Airflow 3, while AIR311 and AIR312 highlight changes that are not currently breaking but are strongly recommended for updates. -The latest available ``ruff`` version will have the most up-to-date rules, but be sure to use at least version ``0.11.6``. The below example demonstrates how to check -for dag incompatibilities that will need to be fixed before they will work as expected on Airflow 3. +The latest available ``ruff`` version will have the most up-to-date rules, but be sure to use at least version ``0.11.13``. The below example demonstrates how to check for dag incompatibilities that will need to be fixed before they will work as expected on Airflow 3. .. code-block:: bash - ruff check dag/ --select AIR301 --preview + ruff check dags/ --select AIR301 --preview To preview the recommended fixes, run the following command: .. code-block:: bash - ruff check dag/ --select AIR301 --show-fixes --preview + ruff check dags/ --select AIR301 --show-fixes --preview Some changes can be automatically fixed. To do so, run the following command: .. code-block:: bash - ruff check dag/ --select AIR301 --fix --preview + ruff check dags/ --select AIR301 --fix --preview +Some of the fixes are marked as unsafe. To trigger these fixes, run the following command: + +.. code-block:: bash + + ruff check dags/ --select AIR301 --fix --unsafe-fixes --preview + +Unsafe fixes usually do not break dag code. They're marked as safe as they may change some runtime behavior. For more information, see `Fix Safety <https://docs.astral.sh/ruff/linter/#fix-safety>`_. Review Comment: Can we move this line above the command? -- 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]
