sul-dlss / dlme-airflow

This is a new repository to capture the work related to the DLME ETL Pipeline and establish airflow
Apache License 2.0
1 stars 0 forks source link

Update package manager section in Readme #553

Closed jacobthill closed 3 weeks ago

jacobthill commented 3 weeks ago

We switched to uv for package management but the Readme still has instructions for poetry. I have a PR here but when I follow those steps I still get this package not found error:

Traceback (most recent call last):
  File "/Users/jtim/Dropbox/Mac/Documents/DLSS/DLME/dlme-airflow/bin/get", line 11, in <module>
    from dlme_airflow.models.provider import Provider
ModuleNotFoundError: No module named 'dlme_airflow'

If I need to do something different, can you put that in the Readme. If I'm doing everything right but you also get this error, can you open another ticket to document it.

edsu commented 3 weeks ago

Thanks for doing this @jacobthill -- what do you run to generate that error?

jacobthill commented 3 weeks ago

@edsu bin/get princeton al_madani > test.json

edsu commented 3 weeks ago

Does uv run bin/get princeton al_madani > test.json work?

Actually that doesn't seem to work for me. I think maybe something is missing from the pyproject.toml?