You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2022/02/19 17:26:32 UTC

[GitHub] [airflow] potiuk edited a comment on pull request #21685: Add extra information about time synchronization needed

potiuk edited a comment on pull request #21685:
URL: https://github.com/apache/airflow/pull/21685#issuecomment-1046066143


   Just one comment - tt's actually very similar reasoning why Tests shoudl be DAMP rather than DRY (DAMP should be favoured): https://testing.googleblog.com/2019/12/testing-on-toilet-tests-too-dry-make.html 
   
   I agree that also in docs  DRY is important but when faced with the dillema DRY or IMMEDIATELY HELPFUL WITHOUT FOLLOWING MORE LINKS (when you see warnings and watchouts) I favour the other. 
   
   BTW.  That gave me an idea - I need to come up with some catchy phrase like DAMP. I think this might be the next content of my blog post and presentation for the summit even :D.


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