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

   ## What
   
   Some example DAGs teach an outdated coding style
   
   ## Why
   
   New users copy these examples to learn Airflow, but 5 of them use the old 
def f(**kwargs): kwargs["ti"] style instead of the modern def f(ti=None): style 
shown elsewhere — inconsistent and confusing for beginners.
   
   ## Tests
   
   All five converted to named parameters, **kwargs dropped where nothing else 
needs it, DAGs behave identically.
   
   <img width="1920" height="929" alt="example_branch_dop_operator_v3" 
src="https://github.com/user-attachments/assets/a24af0d8-769a-42d7-9973-dd1af4246246";
 />
   <img width="1920" height="923" alt="tutorial_objectstore" 
src="https://github.com/user-attachments/assets/54900360-b9e0-4ce1-8244-f99a036ba6eb";
 />
   <img width="1920" height="925" alt="tutorial_dag" 
src="https://github.com/user-attachments/assets/6670becf-b8a2-4174-a22e-3ad662abbe21";
 />
   <img width="1920" height="929" alt="params_ui_tutorial" 
src="https://github.com/user-attachments/assets/c7392496-1391-4ff6-9f21-bd4d638d8e19";
 />
   <img width="1920" height="933" alt="params_trigger_ui" 
src="https://github.com/user-attachments/assets/d6453495-dc23-4d3f-a9a3-b87c87f3f980";
 />
   


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