Skip to content

Comments

[airflow] refactor: remove unnecessary Some in check_method, check_class_attribute (AIR302)#16975

Merged
ntBre merged 2 commits intoastral-sh:mainfrom
astronomer:refactor-AIR302
Mar 26, 2025
Merged

[airflow] refactor: remove unnecessary Some in check_method, check_class_attribute (AIR302)#16975
ntBre merged 2 commits intoastral-sh:mainfrom
astronomer:refactor-AIR302

Conversation

@Lee-W
Copy link
Contributor

@Lee-W Lee-W commented Mar 26, 2025

Summary

remove unnecessary Some

Test Plan

It's a refactoring change. Existing test cases won't be affected

@Lee-W Lee-W changed the title [airflow] refactor: remove uncessary Some in check_method, check_class_attribute (AIR302) [airflow] refactor: remove unnecessary Some in check_method, check_class_attribute (AIR302) Mar 26, 2025
@Lee-W Lee-W force-pushed the refactor-AIR302 branch from 85a3b50 to 6d1ad47 Compare March 26, 2025 07:53
@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 added the internal An internal refactor or improvement label Mar 26, 2025
@ntBre ntBre changed the title [airflow] refactor: remove unnecessary Some in check_method, check_class_attribute (AIR302) [airflow] refactor: remove unnecessary Some in check_method, check_class_attribute (AIR302) Mar 26, 2025
@ntBre ntBre merged commit 58350ec into astral-sh:main Mar 26, 2025
22 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

internal An internal refactor or improvement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants