kevinjqliu opened a new pull request, #2979:
URL: https://github.com/apache/iceberg-python/pull/2979
<!--
Thanks for opening a pull request!
-->
<!-- In the case this PR will resolve an issue, please replace
${GITHUB_ISSUE_ID} below with the actual Github issue id. -->
<!-- Closes #${GITHUB_ISSUE_ID} -->
# Rationale for this change
I noticed running `make install` when a `.venv` already existed would show
an interactive prompt:
```
➜ make install
uv is already installed.
uv venv
Using CPython 3.12.11 interpreter at:
/Users/kevinliu/.pyenv/versions/3.12.11/bin/python3
Creating virtual environment at: .venv
? A virtual environment already exists at `.venv`. Do you want to replace
it? [y/n] › yes
hint: Use the `--clear` flag or set `UV_VENV_CLEAR=1` to skip this prompt
```
We dont need this prompt.
This PR only setup a new venv if it does not exist. And also sync dep only
when necessary.
This should make the entire `make install` process a lot faster and easier
to work with
## Are these changes tested?
Yes running `make install` repeatedly
## Are there any user-facing changes?
<!-- In the case of user-facing changes, please add the changelog label. -->
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
