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

Update tooling, allow environment variables, dockerize #2

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

watzon
Copy link

@watzon watzon commented Oct 19, 2024

This PR does a lot so I'll try to explain it as well as I can.

  1. Add uv as a project manager. This just makes it easier for people to get into the project, assuming they're ok with installing UV. If not they can stick with the normal requirements.txt way of doing things. requirements.txt has also been updated and is serving more as a lockfile for uv, so it now lists all dependencies and has them locked.

  2. I made the config file optional by making it so that you can also use environment variables. This is especially nice when it comes to docker because otherwise you have to mount the config file as a volume.

  3. Added a Dockerfile. This is pretty self explanatory.

  4. Updated the README with a lot of extra instructions, including how to run with Docker.

@ChiefGyk3D
Copy link
Owner

Sweet I will get to testing it this week and see how well it works. Thank you so much for the amazing work

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

Successfully merging this pull request may close these issues.

2 participants