You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@airflow.apache.org by "Calder-Ty (via GitHub)" <gi...@apache.org> on 2023/09/22 20:41:34 UTC

[GitHub] [airflow] Calder-Ty commented on pull request #34564: fix: Make dry_run optional per docs

Calder-Ty commented on PR #34564:
URL: https://github.com/apache/airflow/pull/34564#issuecomment-1732011122

   > IMHO it's a problem in the doc, dry_run should be `True` by default.
   
   I don't care about what the default is, however that change would technically be a breaking change. Albeit, no one can be depending on the declared functionality from the docs as it is broken.
   
   


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org