kevinjqliu commented on PR #1963:
URL: https://github.com/apache/iceberg-python/pull/1963#issuecomment-2855483829
Thanks @petern48 for the contribution and @Fokko for the review :)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to Git
kevinjqliu merged PR #1963:
URL: https://github.com/apache/iceberg-python/pull/1963
--
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...@i
kevinjqliu commented on code in PR #1963:
URL: https://github.com/apache/iceberg-python/pull/1963#discussion_r2075684333
##
.github/workflows/python-ci.yml:
##
@@ -65,4 +65,24 @@ jobs:
- name: Linters
run: make lint
- name: Tests
- run: make test-coverage
+
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
+
Fokko commented on code in PR #1963:
URL: https://github.com/apache/iceberg-python/pull/1963#discussion_r2075001887
##
.github/workflows/python-ci.yml:
##
@@ -65,4 +65,24 @@ jobs:
- name: Linters
run: make lint
- name: Tests
- run: make test-coverage
+
Fokko commented on code in PR #1963:
URL: https://github.com/apache/iceberg-python/pull/1963#discussion_r207593
##
.github/workflows/python-ci.yml:
##
@@ -65,4 +65,24 @@ jobs:
- name: Linters
run: make lint
- name: Tests
- run: make test-coverage
Revie
petern48 commented on code in PR #1963:
URL: https://github.com/apache/iceberg-python/pull/1963#discussion_r2074471301
##
.github/workflows/python-ci.yml:
##
@@ -66,3 +66,20 @@ jobs:
run: make lint
- name: Tests
run: make test-coverage
+
+ integration-test:
+
kevinjqliu commented on code in PR #1963:
URL: https://github.com/apache/iceberg-python/pull/1963#discussion_r2072495090
##
.github/workflows/python-ci.yml:
##
@@ -66,3 +66,20 @@ jobs:
run: make lint
- name: Tests
run: make test-coverage
+
+ integration-test: