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/17 21:12:35 UTC

[GitHub] [airflow] potiuk commented on pull request #20508: Fix dates.days_ago to respect localized timezone configuration.

potiuk commented on pull request #20508:
URL: https://github.com/apache/airflow/pull/20508#issuecomment-1043452068


   I would be against accepting the fix actually. 
   
   I think we should not "fix" things we discourage. That undermines the effort of discouraging it. Actually the sentence from  @Sonins explains it very well:
   
   > I realized days_ago has a problem after I opened this pr so i'm not using it anymore
   
   This is precisely what we wanted to happen. We really others who want to use it this way come here are realize they should not use it. This is a win. 
   
   


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