Skip to content

Conversation

@nailo2c
Copy link
Contributor

@nailo2c nailo2c commented Jun 13, 2025

Hi folks,

When I was studying issue #51320, I found there is an indentation issue that causes the code after the first return False to never be reached.

Hence, I'm submitting this PR. If this was intentional, please let me know.

@nailo2c nailo2c requested a review from uranusjr as a code owner June 13, 2025 16:55
Copy link
Member

@uranusjr uranusjr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice catch

@uranusjr uranusjr added the backport-to-v3-1-test Mark PR with this label to backport to v3-1-test branch label Jun 17, 2025
@uranusjr uranusjr requested a review from dstandish June 17, 2025 06:29
@uranusjr uranusjr added this to the Airflow 3.0.3 milestone Jun 17, 2025
@nailo2c
Copy link
Contributor Author

nailo2c commented Jun 26, 2025

Hi @uranusjr, thanks again for the review! Would you mind helping merge this PR when you get a chance? 😄

Copy link
Contributor

@amoghrajesh amoghrajesh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Well caught!

@uranusjr uranusjr merged commit e55464a into apache:main Jul 2, 2025
56 checks passed
github-actions bot pushed a commit that referenced this pull request Jul 2, 2025
(cherry picked from commit e55464a)

Co-authored-by: Aaron Chen <nailo2c@gmail.com>
@github-actions
Copy link

github-actions bot commented Jul 2, 2025

Backport successfully created: v3-0-test

Status Branch Result
v3-0-test PR Link

github-actions bot pushed a commit to aws-mwaa/upstream-to-airflow that referenced this pull request Jul 2, 2025
(cherry picked from commit e55464a)

Co-authored-by: Aaron Chen <nailo2c@gmail.com>
uranusjr pushed a commit that referenced this pull request Jul 2, 2025
Co-authored-by: Aaron Chen <nailo2c@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport-to-v3-1-test Mark PR with this label to backport to v3-1-test branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants