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

Remove pre-commit-uv and uv from environment definition #450

Closed
16 tasks done
redeboer opened this issue Oct 21, 2024 · 1 comment · Fixed by #452, ComPWA/actions#113, ComPWA/ampform#441, ComPWA/ampform-dpd#144 or ComPWA/benchmarks#26
Labels
🐛 Bug Something isn't working
Milestone

Comments

@redeboer
Copy link
Member

redeboer commented Oct 21, 2024

#407 has resulted in a subtle bug: pre-commit-uv pulls in uv into the virtual environment, which results in caching problems if the environment is installed through a global installation of uv (a global uv installation is required since #333):

error: Failed to generate package metadata for `compwa-github-io==0.0.1 @ editable+.`                                                                                                                                                                                                                                          
  Caused by: Failed to deserialize cache entry
  Caused by: data did not match any variant of untagged enum CacheInfoWire

It should instead be recommended to install pre-commit-uv globally as well once with

uv tool install pre-commit --with pre-commit-uv --force-reinstall

(see these instructions).

Tasks

  1. 🐛 Bug 🖱️ DX
    redeboer
  2. 🔨 Maintenance 🖱️ DX
    redeboer
  3. ⚠️ Interface 🔨 Maintenance 🖱️ DX
    redeboer
  4. 🔨 Maintenance 🖱️ DX
    redeboer
  5. 🔨 Maintenance 🖱️ DX
    redeboer
  6. 🔨 Maintenance 🖱️ DX
    redeboer
  7. 🔨 Maintenance 🖱️ DX
    redeboer
  8. 🔨 Maintenance 🖱️ DX
    redeboer
  9. 🐛 Bug 🔨 Maintenance
    redeboer
  10. ⚠️ Interface 🔨 Maintenance 🖱️ DX
    redeboer
  11. 🔨 Maintenance 🖱️ DX
    redeboer
  12. 🔨 Maintenance 🖱️ DX
    redeboer
  13. 🔨 Maintenance 🖱️ DX
    redeboer
  14. ⚠️ Interface 🔨 Maintenance 🖱️ DX
    redeboer
@redeboer redeboer added the 🐛 Bug Something isn't working label Oct 21, 2024
@redeboer redeboer added this to the 0.5.1 milestone Oct 21, 2024
@redeboer
Copy link
Member Author

Actually, the caching problems may originate from the fact that the version of packages that are installed in editable mode with a dynamic version through setuptools-scm.

The problem pops up irregularly, so I'm not sure yet. It could also be caused by packages that do not have a src/ folder.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment