Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fail removing missing tool #321

Open
spapanik opened this issue Mar 13, 2025 · 4 comments
Open

Fail removing missing tool #321

spapanik opened this issue Mar 13, 2025 · 4 comments

Comments

@spapanik
Copy link

I am experiencing some failures, after I started using setup uv:

https://github.com/spapanik/yamk/actions/runs/13836718332/job/38713514466

The issue with this run is I'm trying to use the tool that I'm trying to test with uv tool. uv tries to remove it from within the venv, but it fails to do so, because it's not present (it's installed with uv tool outside the venv), and therefore installing it fails.

@eifinger
Copy link
Collaborator

Hi.
You said the problems started when you started using this action. Can you link me a run of a working solution before you used this action?

@spapanik
Copy link
Author

Yes, this is the last successful run: https://github.com/spapanik/yamk/actions/runs/12766783858/job/35583816495, where I was managing the venv in CI without the action

@eifinger
Copy link
Collaborator

Looks like the virtual envs are crashing.
Can you please replace setup-uv with manual installation of uv?
I want to make sure its the additional stuff this action does that is causing the problems.

@eifinger
Copy link
Collaborator

What I also see is that in your https://github.com/spapanik/yamk/blob/main/cookbook.yaml you are running uv sync with --frozen but you don't add --frozen to all the uv run statements. uv run calls uv sync under the hood. you should pass your variable ${FROZEN} there too.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants