Skip to content

Comments

[Airflow] Make AIR302 example error out-of-the-box#18988

Merged
ntBre merged 2 commits intoastral-sh:mainfrom
MeGaGiGaGon:patch-4
Jun 30, 2025
Merged

[Airflow] Make AIR302 example error out-of-the-box#18988
ntBre merged 2 commits intoastral-sh:mainfrom
MeGaGiGaGon:patch-4

Conversation

@MeGaGiGaGon
Copy link
Contributor

@MeGaGiGaGon MeGaGiGaGon commented Jun 27, 2025

Summary

Part of #18972

This PR makes airflow3-moved-to-provider (AIR302)'s example error out-of-the-box

Old example

from airflow.auth.managers.fab.fab_auth_manage import FabAuthManager

New example

from airflow.auth.managers.fab.fab_auth_manager import FabAuthManager

fab_auth_manager_app = FabAuthManager().get_fastapi_app()

Test Plan

N/A, no functionality/tests affected

@ntBre ntBre self-requested a review June 27, 2025 17:38
@ntBre ntBre added the documentation Improvements or additions to documentation label Jun 27, 2025
@github-actions
Copy link
Contributor

ruff-ecosystem results

Linter (stable)

✅ ecosystem check detected no linter changes.

Linter (preview)

✅ ecosystem check detected no linter changes.

@ntBre ntBre merged commit 462dbad into astral-sh:main Jun 30, 2025
35 checks passed
iyakushev pushed a commit to iyakushev/ruff that referenced this pull request Jul 1, 2025
<!--
Thank you for contributing to Ruff/ty! To help us out with reviewing,
please consider the following:

- Does this pull request include a summary of the change? (See below.)
- Does this pull request include a descriptive title? (Please prefix
with `[ty]` for ty pull
  requests.)
- Does this pull request include references to any relevant issues?
-->

## Summary

<!-- What's the purpose of the change? What does it do, and why? -->

Part of astral-sh#18972

This PR makes [airflow3-moved-to-provider
(AIR302)](https://docs.astral.sh/ruff/rules/airflow3-moved-to-provider/#airflow3-moved-to-provider-air302)'s
example error out-of-the-box

[Old example](https://play.ruff.rs/1026c008-57bc-4330-93b9-141444f2a611)
```py
from airflow.auth.managers.fab.fab_auth_manage import FabAuthManager
```

[New example](https://play.ruff.rs/b690e809-a81d-4265-9fde-1494caa0b7fd)
```py
from airflow.auth.managers.fab.fab_auth_manager import FabAuthManager

fab_auth_manager_app = FabAuthManager().get_fastapi_app()
```

## Test Plan

<!-- How was it tested? -->

N/A, no functionality/tests affected
@MeGaGiGaGon MeGaGiGaGon deleted the patch-4 branch July 2, 2025 19:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants