Ox0400 commented on code in PR #38582: URL: https://github.com/apache/arrow/pull/38582#discussion_r1398631515
########## python/pyarrow/tests/test_json.py: ########## @@ -17,6 +17,7 @@ from collections import OrderedDict from decimal import Decimal +import os Review Comment: Oh, sorry .... I try `python3.10 -m pip install -e 'dev/archery[all]'` raise this error `ERROR: dev/archery[all] is not a valid editable requirement. It should either be a path to a local project or a VCS URL (beginning with bzr+http, bzr+https, bzr+ssh, bzr+sftp, .....). ` ref: https://arrow.apache.org/docs/developers/python.html#coding-style So I was run `pylint pyarrow/tests/test_json.py` on local not raise the error. I will remove the import. but `flake8 pyarrow/tests/test_json.py` can raise the error `pyarrow/tests/test_json.py:20:1: F401 'os' imported but unused` -- 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]
