Feedback on PR

python/setup.cfg

@@ -42,6 +42,16 @@ package_dir =
     = src
 packages = find:
 python_requires = >=3.7
+install_requires =
+    pytz
+    retrying
+    mmh3
+    fastavro>=1.3.2<1.4.0
+    hmsclient==0.1.1
+    boto3
+    pyarrow
+[options.extras_require]
+dev = tox-travis==0.12; pytest
 
 [options.packages.find]
 where = src

python/src/iceberg/expressions.py

@@ -14,8 +14,6 @@
 # KIND, either express or implied.  See the License for the
 # specific language governing permissions and limitations
 # under the License.
-from __future__ import annotations
-
 from enum import Enum, auto
 
 
@@ -54,7 +52,7 @@ class Operation(Enum):
     AND = auto()
     OR = auto()
 
-    def negate(self) -> Operation:
+    def negate(self) -> "Operation":
         """Returns the operation used when this is negated."""
 
         try:

python/tox.ini

@@ -25,7 +25,6 @@ deps =
     mock
     pytest
     pytest-checkdocs
-    pyarrow
 setenv =
     COVERAGE_FILE = test-reports/{envname}/.coverage
     PYTEST_ADDOPTS = --junitxml=test-reports/{envname}/junit.xml -vv
@@ -51,17 +50,17 @@ deps =
     isort
     autoflake
 commands =
-    autoflake -r --check --ignore-init-module-imports --remove-all-unused-imports setup.py src tests
-    isort --profile black --check-only setup.py src tests
-    black --line-length 130 --check --diff setup.py src tests
+    autoflake -r --check --ignore-init-module-imports --remove-all-unused-imports src tests
+    isort --profile black --check-only src tests
+    black --line-length 130 --check --diff src tests
 
 [testenv:format]
 deps =
     {[testenv:format-check]deps}
 commands =
-    autoflake -r --in-place --ignore-init-module-imports --remove-all-unused-imports setup.py src tests
-    isort --profile black setup.py src tests
-    black --line-length 130 setup.py src tests
+    autoflake -r --in-place --ignore-init-module-imports --remove-all-unused-imports src tests
+    isort --profile black src tests
+    black --line-length 130 src tests
 
 [testenv:type-check]
 deps =