kevinjqliu commented on code in PR #2658:
URL: https://github.com/apache/iceberg-python/pull/2658#discussion_r2463116027


##########
.github/workflows/pypi-build-artifacts.yml:
##########
@@ -68,7 +69,7 @@ jobs:
         env:
           # Ignore 32 bit architectures
           CIBW_ARCHS: "auto64"
-          CIBW_PROJECT_REQUIRES_PYTHON: ">=3.10"
+          CIBW_PROJECT_REQUIRES_PYTHON: ">=3.10,<=3.13"

Review Comment:
   nit: i dont think we need to add an upperbound limit here



##########
pyproject.toml:
##########
@@ -343,6 +347,8 @@ markers = [
 # Turns a warning into an error
 filterwarnings = [
   "error",
+  # Python 3.13 sqlite3 module ResourceWarnings for unclosed database 
connections
+  "ignore:unclosed database in <sqlite3.Connection object*:ResourceWarning",

Review Comment:
   thats interesting, we should fix this for the tests



-- 
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]

Reply via email to