search

pragunbhutani / dbt-llm-tools

RAG based LLM chatbot for dbt projects
Other
48 stars 12 forks source link

updates readthedocs.txt fixes issue #11 #26

Closed codeblech closed 2 weeks ago

codeblech commented 2 weeks ago

fixed readthedocs documentation issue #11

i ran the command poetry export --with docs -f requirements.txt -o requirements-docs.txt on the latest repo checkout. then i ran the build on readthedocs through my personal account, and it worked

pragunbhutani commented 2 weeks ago

Thanks for the fix @codeblech!

Would you like to take this one step further and add a pre-commit hook that checks if the pyproject.toml file has changed and if so, regenerates the requirements-docs.txt file using the command you've mentioned?

We can also do that as a separate issue if you'd rather merge this first, I'm ok with either approach. Let me know!

codeblech commented 2 weeks ago

Ideally, I'd want to merge after setting up the githook. But, we don't currently have a hosted documentation. I believe we should merge as is, and open up a new issue. Either way, I'll coordinate with @ciaokitty and work on it

pragunbhutani commented 2 weeks ago

@codeblech in that case, can you please open an issue / enhancement to add a pre-commit hook so that we don't forget it? Then we can merge this one.

codeblech commented 2 weeks ago

i opened the issue #28 you can add the enhancement label