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

Issues around python in readme instructions for wsl #178

Open
wtimmerman-fitp opened this issue Jan 12, 2025 · 0 comments
Open

Issues around python in readme instructions for wsl #178

wtimmerman-fitp opened this issue Jan 12, 2025 · 0 comments

Comments

@wtimmerman-fitp
Copy link

The issue:
This line in the readme related to wsl:

make build run

Leads to the error:

cd transform && ../.venv/bin/dbt deps
/bin/sh: 1: ../.venv/bin/dbt: not found

Solution:
Seems like the readme could use a few extra lines to prompt folks to sync from the uv lockfile after cloning. I believe you'd just need to add what's below, but I'm just dropping into this project and not sure if there's anything else.

uv venv && uv sync

Another approach may be to make use of that utils/setup_env.sh script in the readme.

A few other things:

These were a bit more tangential to this, but mentioning if someone is mucking around with the above; could open other issues if you'd prefer:

  • Seems like uv python versions (3.11) are a bit inconsistent with what earlier instructions suggest in sudo apt-get install python3.9 python3-pip python3.9-venv. Not sure if it matters given how uv works.
  • In readme, you may want to prompt user to install uv (or incorporate the utils/setup_env.sh script) or install npm, depending on how wordy you want the readme to be. Or maybe just suggest the dev container route first?

Can't wait to try this in earnest! Very cool

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

1 participant