Fokko commented on code in PR #4811:
URL: https://github.com/apache/iceberg/pull/4811#discussion_r876884526
##########
python/src/iceberg/expressions/literals.py:
##########
@@ -44,13 +44,7 @@
TimeType,
UUIDType,
)
-from iceberg.utils.datetime import (
Review Comment:
Hey @dramaticlly @kbendick thanks for the input, and I agree that breaking
it up into multiple lines is more readable.
This changed because I also set isort to a max line length of 130 chars. Now
I've set it to when there are more than 4 imports, it will break it up into a
multi-line statement.
This is configurable by isort, and I've just updated the code accodingly.
Black is totally fine with it, so that should be okay 👍🏻
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
