gemini-code-assist[bot] commented on code in PR #38334: URL: https://github.com/apache/beam/pull/38334#discussion_r3163750992
########## sdks/python/apache_beam/typehints/arrow_type_compatibility.py: ########## @@ -20,6 +20,7 @@ For internal use only, no backward compatibility guarantees. """ +# ruff: noqa: UP006 Review Comment:  The ruff ignore comment is added to suppress UP006, but this file is part of the migration to PEP 585. If the migration is intended to be complete, this ignore should be removed and the code updated to use built-in types. ########## sdks/python/apache_beam/typehints/batch_test.py: ########## @@ -17,6 +17,7 @@ """Unit tests for batched type converters.""" +# ruff: noqa: UP006 Review Comment:  The ruff ignore comment is added to suppress UP006, but this file is part of the migration to PEP 585. If the migration is intended to be complete, this ignore should be removed and the code updated to use built-in types. ########## sdks/python/apache_beam/typehints/decorators.py: ########## @@ -79,6 +79,7 @@ def foo((a, b)): # pytype: skip-file +# ruff: noqa: UP006 Review Comment:  The ruff ignore comment is added to suppress UP006, but this file is part of the migration to PEP 585. If the migration is intended to be complete, this ignore should be removed and the code updated to use built-in types. ########## sdks/python/apache_beam/typehints/native_type_compatibility.py: ########## @@ -19,6 +19,7 @@ # pytype: skip-file +# ruff: noqa: UP006 Review Comment:  The ruff ignore comment is added to suppress UP006, but this file is part of the migration to PEP 585. If the migration is intended to be complete, this ignore should be removed and the code updated to use built-in types. ########## sdks/python/apache_beam/typehints/schemas.py: ########## @@ -67,6 +67,7 @@ # pytype: skip-file +# ruff: noqa: UP006 Review Comment:  The ruff ignore comment is added to suppress UP006, but this file is part of the migration to PEP 585. If the migration is intended to be complete, this ignore should be removed and the code updated to use built-in types. ########## sdks/python/apache_beam/typehints/schemas_test.py: ########## @@ -19,6 +19,7 @@ # pytype: skip-file +# ruff: noqa: UP006 Review Comment:  The ruff ignore comment is added to suppress UP006, but this file is part of the migration to PEP 585. If the migration is intended to be complete, this ignore should be removed and the code updated to use built-in types. ########## sdks/python/apache_beam/typehints/typed_pipeline_test.py: ########## @@ -19,6 +19,7 @@ # pytype: skip-file +# ruff: noqa: UP006 Review Comment:  The ruff ignore comment is added to suppress UP006, but this file is part of the migration to PEP 585. If the migration is intended to be complete, this ignore should be removed and the code updated to use built-in types. ########## sdks/python/apache_beam/typehints/decorators_test.py: ########## @@ -19,6 +19,7 @@ # pytype: skip-file +# ruff: noqa: UP006 Review Comment:  The ruff ignore comment is added to suppress UP006, but this file is part of the migration to PEP 585. If the migration is intended to be complete, this ignore should be removed and the code updated to use built-in types. ########## sdks/python/apache_beam/typehints/row_type.py: ########## @@ -17,6 +17,7 @@ # pytype: skip-file +# ruff: noqa: UP006 Review Comment:  The ruff ignore comment is added to suppress UP006, but this file is part of the migration to PEP 585. If the migration is intended to be complete, this ignore should be removed and the code updated to use built-in types. ########## sdks/python/apache_beam/typehints/native_type_compatibility_test.py: ########## @@ -19,6 +19,7 @@ # pytype: skip-file +# ruff: noqa: UP006 Review Comment:  The ruff ignore comment is added to suppress UP006, but this file is part of the migration to PEP 585. If the migration is intended to be complete, this ignore should be removed and the code updated to use built-in types. ########## sdks/python/apache_beam/typehints/pandas_type_compatibility.py: ########## @@ -51,6 +51,7 @@ compatibility guarantees, except for the type mapping itself. """ +# ruff: noqa: UP006 Review Comment:  The ruff ignore comment is added to suppress UP006, but this file is part of the migration to PEP 585. If the migration is intended to be complete, this ignore should be removed and the code updated to use built-in types. ########## sdks/python/apache_beam/typehints/typecheck_test.py: ########## @@ -23,6 +23,7 @@ # pytype: skip-file +# ruff: noqa: UP006 Review Comment:  The ruff ignore comment is added to suppress UP006, but this file is part of the migration to PEP 585. If the migration is intended to be complete, this ignore should be removed and the code updated to use built-in types. ########## sdks/python/apache_beam/typehints/typehints_test.py: ########## @@ -19,6 +19,7 @@ # pytype: skip-file +# ruff: noqa: UP006 Review Comment:  The ruff ignore comment is added to suppress UP006, but this file is part of the migration to PEP 585. If the migration is intended to be complete, this ignore should be removed and the code updated to use built-in types. ########## sdks/python/apache_beam/typehints/typehints.py: ########## @@ -65,6 +65,7 @@ # pytype: skip-file +# ruff: noqa: UP006 Review Comment:  The ruff ignore comment is added to suppress UP006, but this file is part of the migration to PEP 585. If the migration is intended to be complete, this ignore should be removed and the code updated to use built-in types. -- 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]
