josix commented on code in PR #43383:
URL: https://github.com/apache/airflow/pull/43383#discussion_r1939903694
##########
providers/standard/tests/provider_tests/standard/utils/test_python_virtualenv.py:
##########
@@ -192,25 +192,25 @@ def
test_should_create_virtualenv_with_extra_packages_uv(self, mock_execute_in_s
)
def test_remove_task_decorator(self):
- py_source = '@task.virtualenv(serializer="dill")\ndef f():\nimport
funcsigs'
+ py_source = '@task.virtualenv(serializer="dill")\ndef f():\n import
funcsigs'
Review Comment:
Changed to directly implement the full function for better readability and
to conform to Python syntax. However, arranging the test cases might be a bit
tedious, like writing some dummy decorators. Do you think this would be a good
idea? 🤔
--
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]
