jedcunningham opened a new pull request, #72028:
URL: https://github.com/apache/airflow/pull/72028

   Follow-up cleanup on #68795.
   
   The comments there described how the code used to behave rather than what it 
does now, so they're removed or collapsed to a single line, and the fixtures 
use the `# !problem` marker the rest of the file already uses.
   
   Also drops `test_multiple_nested_non_dag_withs_do_not_exit_dag_context`. 
`is_with_dag_context` is function-local and recomputed on every `visit_With`, 
so extra nested `with`s just take the same branch again — it covered the same 
case as the test above it. The three that remain each pin a different task 
position: after, inside, and outside the nested block.
   
   No behavior change.
   
   ---
   
   ##### Was generative AI tooling used to co-author this PR?
   
   - [X] Yes — Claude Code (Opus 5)
   
   Generated-by: Claude Code (Opus 5) following [the 
guidelines](https://github.com/apache/airflow/blob/main/contributing-docs/05_pull_requests.rst#gen-ai-assisted-contributions)


-- 
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]

Reply via email to