Add Docker support for local development#101
Conversation
srinadhkesineni
commented
May 11, 2025
srinadhkesineni
commented
- Added Dockerfile for containerized setup
- Included .dockerignore
- Updated documentation with Docker usage instructions
docker documentation added
|
This PR adds basic Docker support so users can run the project in an isolated environment. |
|
Thank you! But could you remove the GEMINI_API_KEY? |
|
Thanks for pointing that out! I checked, but I couldn't find any GEMINI_API_KEY in the code. let me know where exactly it's located so I can remove it? |
utils/call_llm.py
Outdated
| # You can comment the previous line and use the AI Studio key instead: | ||
| client = genai.Client( | ||
| api_key=os.getenv("GEMINI_API_KEY", ""), | ||
| api_key=os.getenv("GEMINI_API_KEY", "AIzaSyDzXuRp0hP6wAWUFTGRrUPDiKWffUm7vGk"), |
There was a problem hiding this comment.
I updated the code. could you check that once
|
Any information on how to build and run the docker container in the README with the command that the I would expect to see something like: docker run --rm -it \
-v $(pwd)/output:/output tutorial-codebase-knowledge \
--repo <github_repo> \
--token <github_token>BTW: there is another PR for something similar: https://github.com/The-Pocket/PocketFlow-Tutorial-Codebase-Knowledge/pull/54/files |
Dockerfile
Outdated
|
|
||
| COPY . . | ||
|
|
||
| CMD ["python"] |
There was a problem hiding this comment.
Does this command simply start a Python interpreter in interactive mode?
Should it be something more like CMD ["python", "main.py"...?
There was a problem hiding this comment.
My apologies, but it seems I either forgot to commit the latest changes or they weren't actually committed.
I recommitted the changes now it is working fine. And also this is my first PR so a bit of excitement. could you check once. I updated the readme documentation too.
|
Added a follow up question |
|
@zachary62 hey did you check this PR, it is working fine. |
|
Thank you! |
3 participants
