uranusjr commented on issue #16551: URL: https://github.com/apache/airflow/issues/16551#issuecomment-864864317
> I mean I had (always) used `datetime(2021, 5, 31, tzinfo=timezone.utc)` before and I didn't get this. Not sure when do you mean by “before”, but the `tzinfo.name` call is added three years ago so it’s not exactly recent. But I guess that’s subjective. (FWIW the previous implementation didn’t error because it didn’t checked the timezone at all and was buggy on non-fixed timezones). > I honestly think we should completely get rid of Pendulum. That’s probably a good idea long term, but we’ll need some planning on this to re-write timezone functionalities in scheduling, which are heavily built around Pendulum right now. IMO unifying everything to Peendulum can likely help with the transition *off* Pendulum as well, if we only use the standard stuff (e.g. no `Timezone.name`, only `tzinfo.tzname()`, things like that). -- 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. For queries about this service, please contact Infrastructure at: [email protected]
