dipb-sl commented on PR #70446:
URL: https://github.com/apache/airflow/pull/70446#issuecomment-5161442949

   > Is a CLI command really necessary?
   > 
   > I feel this belongs to lint rules. Eg., Ruff 
[AIR304](https://docs.astral.sh/ruff/rules/airflow3-dag-dynamic-value/) 
captures this exactly
   
   AIR304 is AST based checking so it will not cover all the cases. Simple 
example is: 
   If you store `datetime.now()` in variable and then assign it to `start_time` 
Ruff check will not catch this issue if  assign `datetime.now` directly then 
only it can catch you can verify it.
   


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