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