Re: [PR] Cleanup conftest, remove LocalOutputFile [iceberg-python]

2024-02-25 Thread via GitHub
HonahX merged PR #468: URL: https://github.com/apache/iceberg-python/pull/468 -- 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

Re: [PR] Cleanup conftest, remove LocalOutputFile [iceberg-python]

2024-02-25 Thread via GitHub
kevinjqliu commented on PR #468: URL: https://github.com/apache/iceberg-python/pull/468#issuecomment-1963191731 thanks@HonahX, I just ran `make lint`! -- 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

[PR] Cleanup conftest, remove LocalOutputFile [iceberg-python]

2024-02-24 Thread via GitHub
kevinjqliu opened a new pull request, #468: URL: https://github.com/apache/iceberg-python/pull/468 While browsing `conftest.py`, I noticed `LocalOutputFile` which is not used anywhere. It's first introduced in [this commit](https://github.com/apache/iceberg/commit/98dc1e63d240c32ec7d48e