-
Notifications
You must be signed in to change notification settings - Fork 2.3k
Add automation script to generate mkdocstrings files #1048
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey @seratch the code looks totally fine but I actually don't get why we need this? Why can't the files just be hand-written, takes like 5 seconds lol
@rm-openai indeed, creating a single file takes only a few seconds, but checking which files are missing (like this PR does) could take much longer and automation script can do better job. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes across all projects
This pull request adds a new script for docs, which generates missing `ref/**/*.md` files. The script can be executed when you run `make build-docs` command. The script does not do: - overwrite the existing ones - create files for _XXX.py and `__init__.py` Note that the title part is generated like `tool_context` to `Tool Context`. `openai_provider` will be `Openai Provider`, so some of them needs a little manual work to adjust. The direct need is to add `tool_context.md` for openai#1043 but it should be useful for future maintenance.
This pull request adds a new script for docs, which generates missing
ref/**/*.md
files. The script can be executed when you runmake build-docs
command. The script does not do:__init__.py
Note that the title part is generated like
tool_context
toTool Context
.openai_provider
will beOpenai Provider
, so some of them needs a little manual work to adjust.The direct need is to add
tool_context.md
for #1043 but it should be useful for future maintenance.