petern48 commented on code in PR #1963:
URL: https://github.com/apache/iceberg-python/pull/1963#discussion_r2075659286


##########
.github/workflows/python-ci.yml:
##########
@@ -65,4 +65,24 @@ jobs:
     - name: Linters
       run: make lint
     - name: Tests
-      run: make test-coverage
+      run: make test-coverage-unit
+
+  integration-test:
+    runs-on: ubuntu-22.04
+    strategy:
+      matrix:
+        python: ['3.9', '3.10', '3.11', '3.12']
+
+    steps:
+    - uses: actions/checkout@v4
+      with:
+        fetch-depth: 2

Review Comment:
   This was copy and paste from the old `python-integration.yml` file. I did 
however check on who introduced it in the first place... and it turns out it 
was you at [this 
commit](https://github.com/apache/iceberg-python/commit/b44b00a92c450bf8600cc137274ea964025cb484#diff-d82431956d35d8ce2faf5d9153d0295ff193c20dd4075cadfa980fe6a609d39eR39)
 :)
   
   Shall I remove it? I don't see a reason either?



-- 
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: issues-unsubscr...@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org
For additional commands, e-mail: issues-h...@iceberg.apache.org

Reply via email to