-
Notifications
You must be signed in to change notification settings - Fork 17
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix(server/pandas/filter): Fix datetime object comparisons TCTC-4268 #1524
Conversation
Your Render PR Server URL is https://weaverbird-playground-pr-1524.onrender.com. Follow its progress at https://dashboard.render.com/web/srv-cd1uq41gp3jqar61rclg. |
2386b52
to
c20dd15
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 🔥 !
Some smalls none block comments left !
else: # str | ||
ts = Timestamp(value) | ||
return ts if ts.tzinfo else ts.replace(tz="UTC") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
else: # str | |
ts = Timestamp(value) | |
return ts if ts.tzinfo else ts.replace(tz="UTC") | |
# str | |
ts = Timestamp(value) | |
return ts if ts.tzinfo else ts.replace(tz="UTC") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Linters prefer this, but our style in the various repos is to have an else
branch even when returning in previous branches
c20dd15
to
66ab444
Compare
relates to TCTC-4268 Signed-off-by: Luka Peschke <luka.peschke@toucantoco.com>
Signed-off-by: Luka Peschke <luka.peschke@toucantoco.com>
66ab444
to
7f3821c
Compare
Signed-off-by: Luka Peschke luka.peschke@toucantoco.com