Re: [PR] Add pull-request template [iceberg-python]

2025-03-15 Thread via GitHub
kevinjqliu merged PR #1777: URL: https://github.com/apache/iceberg-python/pull/1777 -- 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

Re: [PR] Add pull-request template [iceberg-python]

2025-03-13 Thread via GitHub
hussein-awala commented on code in PR #1777: URL: https://github.com/apache/iceberg-python/pull/1777#discussion_r1994351796 ## dev/.rat-excludes: ## @@ -1,3 +1,4 @@ +.github/* Review Comment: It's very strange, how `.github/*` worked and `.github/pull_request_template.md` d

Re: [PR] Add pull-request template [iceberg-python]

2025-03-12 Thread via GitHub
Fokko commented on code in PR #1777: URL: https://github.com/apache/iceberg-python/pull/1777#discussion_r1992355841 ## .github/pull_request_template.md: ## Review Comment: Since we don't include the `.github/*` directory in the source release tar, we can ignore this. I don

Re: [PR] Add pull-request template [iceberg-python]

2025-03-11 Thread via GitHub
kevinjqliu commented on PR #1777: URL: https://github.com/apache/iceberg-python/pull/1777#issuecomment-2715103113 Thanks! This should make summarizing the release note easier -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

Re: [PR] Add pull-request template [iceberg-python]

2025-03-11 Thread via GitHub
kevinjqliu commented on code in PR #1777: URL: https://github.com/apache/iceberg-python/pull/1777#discussion_r1989768908 ## .github/pull_request_template.md: ## Review Comment: Need the Apache license headers in the file. -- This is an automated message from the Apache

[PR] Add pull-request template [iceberg-python]

2025-03-08 Thread via GitHub
Fokko opened a new pull request, #1777: URL: https://github.com/apache/iceberg-python/pull/1777 While @kevinjqliu did an amazing job summarizing the new stuff in 0.9.0 in the GitHub release (https://github.com/apache/iceberg-python/releases/tag/pyiceberg-0.9.0), I think it would be good to