Skip to content

[airflow] extend and fix AIR302 rules#15525

Merged
MichaReiser merged 4 commits intoastral-sh:mainfrom
astronomer:extend-air302-rules
Jan 16, 2025
Merged

[airflow] extend and fix AIR302 rules#15525
MichaReiser merged 4 commits intoastral-sh:mainfrom
astronomer:extend-air302-rules

Conversation

@Lee-W
Copy link
Copy Markdown
Contributor

@Lee-W Lee-W commented Jan 16, 2025

Summary

  • fix
    • airflow.secrets.local_filesystem.get_connection should be airflow.secrets.local_filesystemLocalFilesystemBackend.get_connectionairflow.secrets.local_filesystemLocalFilesystemBackend.get_connections
    • airflow.utils.dates.date_range does not have a replacement
  • feat
    • airflow.utils.dag_parsing_context.get_parsing_context -> airflow.sdk.get_parsing_context

Test Plan

a test fixture has been updated

@github-actions
Copy link
Copy Markdown
Contributor

ruff-ecosystem results

Linter (stable)

✅ ecosystem check detected no linter changes.

Linter (preview)

✅ ecosystem check detected no linter changes.

@Lee-W Lee-W marked this pull request as draft January 16, 2025 08:28
@Lee-W Lee-W changed the title fix(AIR302): fix wrong airflow.utils.dates.date_range replacement [airflow] extend and fix AIR302 rules Jan 16, 2025
@Lee-W Lee-W marked this pull request as ready for review January 16, 2025 09:09
@MichaReiser MichaReiser added the rule Implementing or modifying a lint rule label Jan 16, 2025
@MichaReiser MichaReiser merged commit fc9dd63 into astral-sh:main Jan 16, 2025
@MichaReiser MichaReiser added the preview Related to preview mode features label Jan 16, 2025
@Lee-W Lee-W deleted the extend-air302-rules branch January 23, 2025 08:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

preview Related to preview mode features rule Implementing or modifying a lint rule

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants