ecerulm commented on issue #16551: URL: https://github.com/apache/airflow/issues/16551#issuecomment-864836448
> I think the timezone argument was really designed to only except pendulum.tz.Timezone, not datetime.timezone (or dateutil.tz.* for that matter). I can't guarantee it but I think this is a regression, I mean I had (always) used `datetime(2021, 5, 31, tzinfo=timezone.utc)` before and I didn't get this. But anyway I agree that you that probably you can quickly solve this particular issue and prevent lots of confused users by `start_date = pendulum.instance(start_date)`, etc in the constructors that take datetimes. If checked and converting to pendulum datetime provides a `tzinfo` with a `name`: ``` pendulum.instance(datetime(2021, 5, 31, tzinfo=timezone.utc)).tzinfo.name # '+00:00' ``` -- 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]
